Skip to content
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

update version and change logs for release 1.1.6 #103

Merged
merged 6 commits into from
Feb 10, 2025
Merged

update version and change logs for release 1.1.6 #103

merged 6 commits into from
Feb 10, 2025

Conversation

gq1
Copy link
Member

@gq1 gq1 commented Feb 6, 2025

PR checklist

  • This comment contains a description of changes (with reason).
  • If you've fixed a bug or added code that should be tested, add tests!
  • If you've added a new tool - have you followed the pipeline conventions in the contribution docs
  • Make sure your code lints (nf-core lint).
  • Ensure the test suite passes (nextflow run . -profile test,docker --outdir <OUTDIR>).
  • Usage Documentation in docs/usage.md is updated.
  • Output Documentation in docs/output.md is updated.
  • CHANGELOG.md is updated.
  • README.md is updated (including new tool citations and authors/contributors).

@gq1 gq1 requested a review from muffato February 6, 2025 11:53
Copy link

github-actions bot commented Feb 6, 2025

nf-core lint overall result: Passed ✅ ⚠️

Posted for pipeline commit 9a5157a

+| ✅ 125 tests passed       |+
#| ❔  26 tests were ignored |#
!| ❗   1 tests had warnings |!

❗ Test warnings:

❔ Tests ignored:

  • files_exist - File is ignored: CODE_OF_CONDUCT.md
  • files_exist - File is ignored: assets/nf-core-variantcalling_logo_light.png
  • files_exist - File is ignored: docs/images/nf-core-variantcalling_logo_light.png
  • files_exist - File is ignored: docs/images/nf-core-variantcalling_logo_dark.png
  • files_exist - File is ignored: .github/ISSUE_TEMPLATE/config.yml
  • files_exist - File is ignored: .github/workflows/awstest.yml
  • files_exist - File is ignored: .github/workflows/awsfulltest.yml
  • files_exist - File is ignored: assets/multiqc_config.yml
  • files_exist - File is ignored: conf/igenomes.config
  • nextflow_config - Config variable ignored: manifest.name
  • nextflow_config - Config variable ignored: manifest.homePage
  • files_unchanged - File ignored due to lint config: LICENSE or LICENSE.md or LICENCE or LICENCE.md
  • files_unchanged - File ignored due to lint config: .github/CONTRIBUTING.md
  • files_unchanged - File ignored due to lint config: .github/ISSUE_TEMPLATE/bug_report.yml
  • files_unchanged - File does not exist: .github/ISSUE_TEMPLATE/config.yml
  • files_unchanged - File ignored due to lint config: .github/PULL_REQUEST_TEMPLATE.md
  • files_unchanged - File ignored due to lint config: .github/workflows/branch.yml
  • files_unchanged - File ignored due to lint config: .github/workflows/linting.yml
  • files_unchanged - File ignored due to lint config: assets/email_template.html
  • files_unchanged - File ignored due to lint config: assets/sendmail_template.txt
  • files_unchanged - File does not exist: assets/nf-core-variantcalling_logo_light.png
  • files_unchanged - File does not exist: docs/images/nf-core-variantcalling_logo_light.png
  • files_unchanged - File does not exist: docs/images/nf-core-variantcalling_logo_dark.png
  • files_unchanged - File ignored due to lint config: lib/NfcoreTemplate.groovy
  • actions_awstest - 'awstest.yml' workflow not found: /home/runner/work/variantcalling/variantcalling/.github/workflows/awstest.yml
  • multiqc_config - 'assets/multiqc_config.yml' not found

✅ Tests passed:

Run details

  • nf-core/tools version 2.8
  • Run at 2025-02-08 18:52:12

@gq1 gq1 requested a review from tkchafin February 6, 2025 12:43
@muffato
Copy link
Member

muffato commented Feb 6, 2025

Oh, the test fails ! The pipeline is making a minimap command with -I-0.0G. I knew that floating point values can be tricky, but never saw this one.

@muffato
Copy link
Member

muffato commented Feb 8, 2025

All the tests ran fine for me on the farm in my default setup. However I could reproduce the problem by using an older version of Nextflow, e.g. 22.10
The problem seems to come from this line:

.map { fasta -> [ [ 'id': fasta.baseName - ~/.fa\w*$/ , 'genome_size': fasta.size() ], fasta ] }

On Nextflow 22.10 and 23.04, fasta.size() returns -1. On Nextflow 23.10 and later, it returns the actual file size. I can't really recognise anything related in the changelog, but it feels like a bug they eventually fixed.
I'm still surprised we didn't spot the problem earlier as this line of code is essentially copied from readmapping which also runs 22.10 on CI and sets nextflowVersion = '!>=22.10.1'

The simplest change here would be to run the CI on 23.10. Since Nextflow and nf-core are pushing changes that require Nextflow 24.04, I don't think it's very limiting anyway. What do you think ?

gq1 added 4 commits February 8, 2025 18:14
Update the minimum supported nextflow version to 23.10.1
minimum nextflow version change
@gq1
Copy link
Member Author

gq1 commented Feb 8, 2025

@sanger-tolsoft fix linting

@gq1 gq1 merged commit a43152a into dev Feb 10, 2025
6 checks passed
@gq1 gq1 deleted the Release1.1.6 branch February 11, 2025 12:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants