Skip to content

Post release bump#332

Merged
jfy133 merged 2 commits intodevfrom
post-patch-bump
Aug 25, 2022
Merged

Post release bump#332
jfy133 merged 2 commits intodevfrom
post-patch-bump

Conversation

@jfy133
Copy link
Copy Markdown
Member

@jfy133 jfy133 commented Aug 25, 2022

For dev

@jfy133 jfy133 requested review from d4straub and skrakau August 25, 2022 10:41
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Aug 25, 2022

nf-core lint overall result: Passed ✅ ⚠️

Posted for pipeline commit 77b9842

+| ✅ 147 tests passed       |+
#| ❔   1 tests were ignored |#
!| ❗   1 tests had warnings |!
Details

❗ Test warnings:

  • readme - README did not have a Nextflow minimum version badge.

❔ Tests ignored:

  • files_unchanged - File ignored due to lint config: lib/NfcoreTemplate.groovy

✅ Tests passed:

Run details

  • nf-core/tools version 2.4.1
  • Run at 2022-08-25 10:54:52

Copy link
Copy Markdown
Collaborator

@d4straub d4straub left a comment

Choose a reason for hiding this comment

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

Typically we hope to not need to release another bug fix, i.e. 2.2.2, but rather a feature release, i.e. 2.3.0, I guess. No idea how that is handled usually. Not so important anyway.

Comment thread CHANGELOG.md
Comment thread nextflow.config Outdated
Co-authored-by: Daniel Straub <[email protected]>
@jfy133
Copy link
Copy Markdown
Member Author

jfy133 commented Aug 25, 2022

Ok, lets keep CHANGELOG as dev (as that's how it's done by edfault AFAIK), but bump to 2.3.0 fine with me.

I actually just remembered that nf-core is meant to use a special patch branch for actual branches...

Maybe we should use that in the future!

https://github.com/nf-core/nf-co.re/blob/1caada5226c0969154df26ffc56fafa7904e674d/markdown/developers/tutorials/nf_core_contributing_overview.md#branch-protection

@jfy133 jfy133 merged commit 0a376a9 into dev Aug 25, 2022
@jfy133 jfy133 deleted the post-patch-bump branch August 25, 2022 11:58
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.

2 participants