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

Release 2.4.1 #260

Merged
merged 36 commits into from
Apr 6, 2023
Merged

Release 2.4.1 #260

merged 36 commits into from
Apr 6, 2023

Conversation

jonasscheid
Copy link
Collaborator

@jonasscheid jonasscheid commented Apr 4, 2023

Added

Fixed

  • Increased comet search, through altering the spectrum_batch_size from 500 to 0
  • #249 - nf-core template update (version 2.7.2) @marissaDubbelaar
  • #258 - Adjusted decoy strategy to reverse #255 and made consistent fdr-level flags #228
  • #845 - Adjusted nf-core test data set #233

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- [ ] If necessary, also make a PR on the nf-core/mhcquant branch on the nf-core/test-datasets repository.
  • 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).

jonasscheid and others added 30 commits November 22, 2022 16:22
Important! Template update for nf-core/tools v2.7.2
…'null' folder creation and update the CHANGELOG
Added low resolution settings to pipeline
@github-actions
Copy link

github-actions bot commented Apr 4, 2023

nf-core lint overall result: Passed ✅ ⚠️

Posted for pipeline commit f7ddde2

+| ✅ 157 tests passed       |+
!| ❗   1 tests had warnings |!

❗ Test warnings:

✅ Tests passed:

Run details

  • nf-core/tools version 2.7.2
  • Run at 2023-04-05 11:57:44

Copy link
Contributor

@steffenlem steffenlem left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link

@ramprasadn ramprasadn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! 👍🏻 🚀

@jonasscheid jonasscheid merged commit 3feaf7c into master Apr 6, 2023
nextflowVersion = '!>=21.10.3'
version = '2.3.2dev'
nextflowVersion = '!>=22.10.1'
version = '2.4.1dev'
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

dev needs to be removed before the release 🙈

@@ -3,6 +3,23 @@
The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/)
and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).

## v2.4.1 nfcore/mhcquant "Maroon Gold Boxer" (patch) - 2023/04/04
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the release date is not up-to-date anymore

@FriederikeHanssen
Copy link
Contributor

Also just saw that in some places exit is still used. would be good to remove that and replace with Nextflow.error in the .groovy files and error in .nf files

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.

6 participants