Skip to content

Update usage.md - #1003

Merged
d4straub merged 2 commits into
nf-core:devfrom
ppflrs:patch-1
May 18, 2026
Merged

Update usage.md#1003
d4straub merged 2 commits into
nf-core:devfrom
ppflrs:patch-1

Conversation

@ppflrs

@ppflrs ppflrs commented May 13, 2026

Copy link
Copy Markdown
Contributor

Fixed typo (L248: contamiation -> contamination) on usage.md

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/ampliseq branch on the nf-core/test-datasets repository.
  • Make sure your code lints (nf-core pipelines lint).
  • Ensure the test suite passes (nextflow run . -profile test,docker --outdir <OUTDIR>).
  • Check for unexpected warnings in debug mode (nextflow run . -profile debug,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).

d4straub and others added 2 commits April 15, 2026 18:13
Fixed typo (L248: contamiation -> contamination) on usage.md
@github-actions

github-actions Bot commented May 13, 2026

Copy link
Copy Markdown

This PR is against the master branch ❌

  • Do not close this PR
  • Click Edit and change the base to dev
  • This CI test will remain failed until you push a new commit

Hi @ppflrs,

It looks like this pull-request is has been made against the ppflrs/ampliseq master branch.
The master branch on nf-core repositories should always contain code from the latest release.
Because of this, PRs to master are only allowed if they come from the ppflrs/ampliseq dev branch.

You do not need to close this PR, you can change the target branch to dev by clicking the "Edit" button at the top of this page.
Note that even after this, the test will continue to show as failing until you push a new commit.

Thanks again for your contribution!

@d4straub

Copy link
Copy Markdown
Collaborator

Thanks! However, we are not working on the master branch but the dev branch. Could you please base the change on the dev branch and open a PR to dev? You are welcome to change that directly here in this PR, of course.

@ppflrs
ppflrs changed the base branch from master to dev May 15, 2026 15:45
@ppflrs

ppflrs commented May 15, 2026

Copy link
Copy Markdown
Contributor Author

Thanks! However, we are not working on the master branch but the dev branch. Could you please base the change on the dev branch and open a PR to dev? You are welcome to change that directly here in this PR, of course.

Hi Daniel, thanks for the heads-up I have changed the branch to dev. I hope this works. Thanks!

@d4straub d4straub left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thanks!

@d4straub
d4straub merged commit e2a4743 into nf-core:dev May 18, 2026
5 of 7 checks passed
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