-
Notifications
You must be signed in to change notification settings - Fork 842
Dev -> Master for 3.16.1 #1413
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Dev -> Master for 3.16.1 #1413
Changes from 73 commits
Commits
Show all changes
76 commits
Select commit
Hold shift + click to select a range
8af71e7
nf-core pipelines bump-version 3.17.0dev
maxulysse 26901ea
Bump version in CHANGELOG
maxulysse 3d00806
Bump versions in snap
maxulysse 14f0ab4
Template update for nf-core/tools version 3.0.0
nf-core-bot 224655a
Merge pull request #1398 from maxulysse/3.17.0dev
maxulysse db6dfd0
Template update for nf-core/tools version 3.0.1
nf-core-bot 2f94e7a
Merge branch 'dev' into nf-core-template-merge-3.0.1
maxulysse 0dbd32a
fix merge conflicts
maxulysse ee83375
Apply suggestions from code review
maxulysse 35b36db
fix nf-core lint
maxulysse b47c526
install nf-test 0.9.0
maxulysse fc80d92
install nf-test once only
maxulysse a24e284
use proper Channel factory
maxulysse 76a1375
rename function test
maxulysse 382189c
fix align
maxulysse 8944480
fix samplesheet usage
maxulysse c71d103
fix tests
maxulysse 35a94de
fix tests
maxulysse 76c5616
update CHANGELOG
maxulysse 452850a
Apply suggestions from code review
maxulysse 99eeff0
update test adjacent files
maxulysse 6ed6014
clean nftignore for kallisto
maxulysse a69f1cb
minor code polish
maxulysse 50d4796
kallisto: update snapshots
maxulysse 1acae78
update plugin version
maxulysse 8d776ac
update nftignore
maxulysse a94823a
update fastq_fastqc_umitools_trimgalore
maxulysse 711e66d
update nftignore and snapshots for salmon and kallisto
maxulysse 4198a88
no set -C until we fix modules upstream
maxulysse b2ac86f
update nftignore
maxulysse 2ce830b
update default tests
maxulysse e5c8c9d
update featurecounts_group_type tests
maxulysse 20e5e7d
update nftignore and hisat2 snapshots
maxulysse 997fb25
update min_mapped_reads snapshots
maxulysse 6866c24
do not snapshot the bam
maxulysse 0e2fe1a
update remove_ribo_rna tests
maxulysse 60f73b9
update skip_qc tests
maxulysse 7629795
update skip_trimming snapshots
maxulysse 1dcfeb2
Update subworkflow to fix variable
pinin4fjords 38a4147
update changelog
pinin4fjords a93e9bb
update last snapshots
maxulysse 386d6bd
ignore file
maxulysse ea70b71
Merge pull request #1401 from nf-core/nf-core-template-merge-3.0.1
maxulysse 792aa12
Merge branch 'dev' into fix_swf_variable
maxulysse f959198
Correct CHANGELOG
pinin4fjords fab468e
Merge branch 'fix_swf_variable' of github.com:nf-core/rnaseq into fix…
pinin4fjords d40e650
only one samplesheetToList
maxulysse 3989b24
update CHANGELOG
maxulysse a044cd3
Template update for nf-core/tools version 3.0.2
nf-core-bot 94f2c38
Merge branch 'dev' into nf-core-template-merge-3.0.2
maxulysse 3aaa983
fix GHA
maxulysse 9514d5f
fix GHA
maxulysse 8be89a3
fix GHA
maxulysse f44e93a
fix snapshots
maxulysse 4b46c7b
Merge pull request #1407 from nf-core/nf-core-template-merge-3.0.2
maxulysse 1879afa
Merge branch 'dev' into fix_swf_variable
maxulysse bf0a3c1
Merge branch 'dev' into improve_samplesheetToList
maxulysse 08dbcc9
Merge pull request #1405 from nf-core/fix_swf_variable
maxulysse 141c15a
Merge branch 'dev' into improve_samplesheetToList
maxulysse 7794c69
Merge pull request #1406 from maxulysse/improve_samplesheetToList
maxulysse e5d128d
:shakefist:
maxulysse 95b2927
Fix ifEmpty() issues with trimming
pinin4fjords f6355ee
Update CHANGELOG
pinin4fjords a391518
Update rnaseq prepro
pinin4fjords 660d10c
Merge pull request #1410 from nf-core/fix_empty_trimming
maxulysse 56aef9b
Merge pull request #1409 from maxulysse/fix_manifest_display_CLI
maxulysse 8bd139b
Reset versions back to 3.16.1 for patch release
pinin4fjords c32cbd9
Update changelog
pinin4fjords 7e56935
Merge branch 'dev' into prerelease_3.16.1
pinin4fjords a32cd19
Merge pull request #1412 from nf-core/prerelease_3.16.1
pinin4fjords 7524cf0
disable conda pipeline level tests + update CHANGELOG
maxulysse 8c13b13
fix PR link
maxulysse 4061de3
Merge pull request #1415 from maxulysse/disable_pipeline_conda_tests
pinin4fjords 7bb5400
Properly disable conda pipeline level tests
maxulysse acae3f7
Update CHANGELOG.md
maxulysse 773762a
Merge pull request #1416 from nf-core/maxulysse-patch-1
maxulysse File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,6 +1,6 @@ | ||
| name: nf-core linting | ||
| # This workflow is triggered on pushes and PRs to the repository. | ||
| # It runs the `nf-core lint` and markdown lint tests to ensure | ||
| # It runs the `nf-core pipelines lint` and markdown lint tests to ensure | ||
| # that the code meets the nf-core guidelines. | ||
| on: | ||
| push: | ||
|
|
@@ -41,17 +41,32 @@ jobs: | |
| python-version: "3.12" | ||
| architecture: "x64" | ||
|
|
||
| - name: read .nf-core.yml | ||
| uses: pietrobolcato/[email protected] | ||
| id: read_yml | ||
| with: | ||
| config: ${{ github.workspace }}/.nf-core.yml | ||
|
|
||
| - name: Install dependencies | ||
| run: | | ||
| python -m pip install --upgrade pip | ||
| pip install nf-core | ||
| pip install nf-core==${{ steps.read_yml.outputs['nf_core_version'] }} | ||
|
|
||
| - name: Run nf-core pipelines lint | ||
| if: ${{ github.base_ref != 'master' }} | ||
| env: | ||
| GITHUB_COMMENTS_URL: ${{ github.event.pull_request.comments_url }} | ||
| GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }} | ||
| GITHUB_PR_COMMIT: ${{ github.event.pull_request.head.sha }} | ||
| run: nf-core -l lint_log.txt pipelines lint --dir ${GITHUB_WORKSPACE} --markdown lint_results.md | ||
|
|
||
| - name: Run nf-core lint | ||
| - name: Run nf-core pipelines lint --release | ||
| if: ${{ github.base_ref == 'master' }} | ||
| env: | ||
| GITHUB_COMMENTS_URL: ${{ github.event.pull_request.comments_url }} | ||
| GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }} | ||
| GITHUB_PR_COMMIT: ${{ github.event.pull_request.head.sha }} | ||
| run: nf-core -l lint_log.txt lint --dir ${GITHUB_WORKSPACE} --markdown lint_results.md | ||
| run: nf-core -l lint_log.txt pipelines lint --release --dir ${GITHUB_WORKSPACE} --markdown lint_results.md | ||
|
|
||
| - name: Save PR number | ||
| if: ${{ always() }} | ||
|
|
||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,46 @@ | ||
| name: nf-core template version comment | ||
| # This workflow is triggered on PRs to check if the pipeline template version matches the latest nf-core version. | ||
| # It posts a comment to the PR, even if it comes from a fork. | ||
|
|
||
| on: pull_request_target | ||
|
|
||
| jobs: | ||
| template_version: | ||
| runs-on: ubuntu-latest | ||
| steps: | ||
| - name: Check out pipeline code | ||
| uses: actions/checkout@0ad4b8fadaa221de15dcec353f45205ec38ea70b # v4 | ||
| with: | ||
| ref: ${{ github.event.pull_request.head.sha }} | ||
|
|
||
| - name: Read template version from .nf-core.yml | ||
| uses: nichmor/[email protected] | ||
| id: read_yml | ||
| with: | ||
| config: ${{ github.workspace }}/.nf-core.yml | ||
|
|
||
| - name: Install nf-core | ||
| run: | | ||
| python -m pip install --upgrade pip | ||
| pip install nf-core==${{ steps.read_yml.outputs['nf_core_version'] }} | ||
|
|
||
| - name: Check nf-core outdated | ||
| id: nf_core_outdated | ||
| run: echo "OUTPUT=$(pip list --outdated | grep nf-core)" >> ${GITHUB_ENV} | ||
|
|
||
| - name: Post nf-core template version comment | ||
| uses: mshick/add-pr-comment@b8f338c590a895d50bcbfa6c5859251edc8952fc # v2 | ||
| if: | | ||
| contains(env.OUTPUT, 'nf-core') | ||
| with: | ||
| repo-token: ${{ secrets.NF_CORE_BOT_AUTH_TOKEN }} | ||
| allow-repeats: false | ||
| message: | | ||
| > [!WARNING] | ||
| > Newer version of the nf-core template is available. | ||
| > | ||
| > Your pipeline is using an old version of the nf-core template: ${{ steps.read_yml.outputs['nf_core_version'] }}. | ||
| > Please update your pipeline to the latest version. | ||
| > | ||
| > For more documentation on how to update your pipeline, please see the [nf-core documentation](https://github.com/nf-core/tools?tab=readme-ov-file#sync-a-pipeline-with-the-template) and [Synchronisation documentation](https://nf-co.re/docs/contributing/sync). | ||
| # |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -10,3 +10,4 @@ test.xml | |
| testing* | ||
| testing/ | ||
| work/ | ||
| null/ | ||
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.