-
Notifications
You must be signed in to change notification settings - Fork 842
Important! Template update for nf-core/tools v3.0.1 #1401
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
Merged
Changes from 4 commits
Commits
Show all changes
36 commits
Select commit
Hold shift + click to select a range
14f0ab4
Template update for nf-core/tools version 3.0.0
nf-core-bot 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 a93e9bb
update last snapshots
maxulysse 386d6bd
ignore file
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,43 @@ | ||
| 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 | ||
|
|
||
| - name: Read template version from .nf-core.yml | ||
| uses: pietrobolcato/[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: pip list --outdated | grep nf-core | ||
|
|
||
| - name: Post nf-core template version comment | ||
| uses: mshick/add-pr-comment@b8f338c590a895d50bcbfa6c5859251edc8952fc # v2 | ||
| if: | | ||
| ${{ steps.nf_core_outdated.outputs.stdout }} =~ '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
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,15 +1,29 @@ | ||
| repository_type: pipeline | ||
| nf_core_version: "2.14.1" | ||
| bump_version: null | ||
| lint: | ||
| actions_ci: false | ||
| files_exist: | ||
| - conf/modules.config | ||
| files_unchanged: | ||
| - assets/email_template.html | ||
| - assets/email_template.txt | ||
| - .gitignore | ||
| files_exist: | ||
| - conf/modules.config | ||
| actions_ci: false | ||
| multiqc_config: false | ||
| modules_config: false | ||
| multiqc_config: false | ||
| nextflow_config: | ||
| - config_defaults: | ||
| - params.ribo_database_manifest | ||
| nf_core_version: 3.0.1 | ||
| org_path: null | ||
| repository_type: pipeline | ||
| template: | ||
| author: "Harshil Patel, Phil Ewels, Rickard Hammar\xE9n" | ||
| description: RNA sequencing analysis pipeline for gene/isoform quantification and | ||
| extensive quality control. | ||
| force: false | ||
| is_nfcore: true | ||
| name: rnaseq | ||
| org: nf-core | ||
| outdir: . | ||
| skip_features: null | ||
| version: 3.17.0dev | ||
| update: null |
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 |
|---|---|---|
|
|
@@ -7,7 +7,7 @@ repos: | |
| - [email protected] | ||
|
|
||
| - repo: https://github.com/editorconfig-checker/editorconfig-checker.python | ||
| rev: "2.7.3" | ||
| rev: "3.0.3" | ||
| hooks: | ||
| - id: editorconfig-checker | ||
| alias: ec | ||
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
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.