Skip to content

Staramr/update/0.12.3 - #227

Merged
sgsutcliffe merged 10 commits into
devfrom
staramr/update/0.12.3
Aug 12, 2026
Merged

Staramr/update/0.12.3#227
sgsutcliffe merged 10 commits into
devfrom
staramr/update/0.12.3

Conversation

@sgsutcliffe

Copy link
Copy Markdown
Contributor

Updating staramr moduled container to use 0.12.3 staramr

@apetkau apetkau left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Approved with a few minor comments.

Comment thread .nf-core.yml
name: mikrokondo
description: Mikrokondo Pipeline
version: 0.10.1
version: 0.10.2

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Could you bump this to 0.10.3 so it matches the other version numbers.

Comment thread CHANGELOG.md
and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).

## Unreleased
## 0.10.3

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Could you break up the changes into an ### Updated and ### Added subsections?

Also, could you move any of the references to PR 220 (staramr updates) out of the 0.10.2 section and under the 0.10.3 section (and remove any duplicate statements if there are any).

Comment thread CHANGELOG.md

- Added StarAMR command-line parameters as nextflow parameters, including species specific parameter settings. [PR 220](https://github.com/phac-nml/mikrokondo/pull/220)

## 0.10.2

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

0.10.2 was released on March 12 (https://github.com/phac-nml/mikrokondo/releases/tag/0.10.2). Could you add the date after the version number here so it matches the other releases?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Just realized this got properly messed up. Might have been some merge conflicts badly handled

@sgsutcliffe
sgsutcliffe merged commit 8ab5eaa into dev Aug 12, 2026
17 checks passed
@sgsutcliffe
sgsutcliffe deleted the staramr/update/0.12.3 branch August 12, 2026 13:10
@sgsutcliffe sgsutcliffe mentioned this pull request Aug 12, 2026
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