Skip to content

Commit

Permalink
revert JDKv11
Browse files Browse the repository at this point in the history
  • Loading branch information
drernie committed Oct 23, 2024
1 parent ccbbd9c commit bfeea21
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 2 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@ jobs:
fail-fast: false
matrix:
os: [windows-latest, ubuntu-latest, macos-latest]
java_version: [17, 19]
java_version: [11, 17, 19]
runs-on: ${{ matrix.os }}

steps:
Expand Down
1 change: 0 additions & 1 deletion CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,6 @@
- Use package cache instead of `params` to find output URIs
(in order to support dynamic URIs set by, e.g. `main.nf`)
- Allow setting metadata from inside the workflow
- Dropped support for JDK 11 on Windows, deprecated on other platforms

## [0.8.6] 2024-09-11

Expand Down
3 changes: 3 additions & 0 deletions plugins/nf-quilt/src/main/nextflow/quilt/QuiltProduct.groovy
Original file line number Diff line number Diff line change
Expand Up @@ -163,6 +163,7 @@ ${nextflow}
}

boolean shouldSkip(key) {
print("shouldSkip[$key]: ${pkg.meta}")
return pkg.meta.containsKey(key) && pkg.meta[key] == KEY_SKIP
}

Expand All @@ -172,6 +173,7 @@ ${nextflow}
}
try {
Map smeta = getMetadata(session.config)
println("addSessionMeta.smeta: ${smeta}")
smeta['quilt'] = [package_id: pkg.toString(), uri: path.toUriString()]
msg = "${smeta['config']['runName']}: ${smeta['cmd']}"
smeta.remove('config')
Expand All @@ -180,6 +182,7 @@ ${nextflow}
log.error("addSessionMeta.getMetadata failed: ${e.getMessage()}", pkg.meta)
return false
}
println("addSessionMeta.meta: ${meta}")
writeNextflowMetadata(meta, 'metadata')
return true
}
Expand Down

0 comments on commit bfeea21

Please sign in to comment.