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

Addition of zenodo doi #27

Merged
merged 4 commits into from
Apr 4, 2024
Merged

Addition of zenodo doi #27

merged 4 commits into from
Apr 4, 2024

Conversation

jannikseidelQBiC
Copy link
Collaborator

PR checklist

  • This comment contains a description of changes (with reason).
  • README.md is updated (including new tool citations and authors/contributors).

Addition of zenodo doi

To finalize the initial release and to make dev up-to-date with master this PR is created.

README.md Outdated
@@ -82,8 +82,7 @@ For further information or help, don't hesitate to get in touch on the [Slack `#

## Citations

<!-- TODO nf-core: Add citation for pipeline after first release. Uncomment lines below and update Zenodo doi and badge at the top of this file. -->
<!-- If you use nf-core/detaxizer for your analysis, please cite it using the following doi: [10.5281/zenodo.XXXXXX](https://doi.org/10.5281/zenodo.XXXXXX) -->
If you use nf-core/detaxizer for your analysis, please cite it using the following doi: [10.5281/zenodo.XXXXXX](https://doi.org/10.5281/zenodo.10877147)
Copy link
Contributor

Choose a reason for hiding this comment

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

Hm, wouldnt be

Suggested change
If you use nf-core/detaxizer for your analysis, please cite it using the following doi: [10.5281/zenodo.XXXXXX](https://doi.org/10.5281/zenodo.10877147)
If you use nf-core/detaxizer for your analysis, please cite it using the following doi: [10.5281/zenodo.10877147](https://doi.org/10.5281/zenodo.10877147)

nicer?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@FriederikeHanssen Could you update this on master, please? 🙂 And thanks for the support in the release!

@FriederikeHanssen
Copy link
Contributor

Added the doi directly on master. You need to merge master -> dev to get the latest changes. This PR can be closed.

Copy link
Contributor

@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.

LGTM

@jannikseidelQBiC jannikseidelQBiC merged commit 4bb09be into dev Apr 4, 2024
7 of 10 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.

3 participants