-
Notifications
You must be signed in to change notification settings - Fork 4
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 1.0.0 #7
Conversation
(e.g. nf-core github actions)
Important! Template update for nf-core/tools v2.14.1
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
see #8 (review)
Added my thoughts #8 (review) |
…essing results, some minor adjustments in the docs.
Co-authored-by: Matthias Hörtenhuber <[email protected]>
Co-authored-by: Matthias Hörtenhuber <[email protected]>
Adressing review suggestions for Version 1.0.0
Adressing more reviewer comments for 1.0.0
Disabled publishing of version.yml in visualization processes
Template update for nf-core/tools version 3.1.1
workflows/rangeland.nf
Outdated
} | ||
|
||
data.ifEmpty { | ||
error "No directories found!" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would maybe make this more informative.
error "No directories found!" | |
error "[nf-core/rangeland] ERROR: No directories found in input tar file!" |
Or similar
My approval still stands! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some last comment over at #8 but good to go otherwise
Addressed final reviewer suggestion for version 1.0.0
PR description
This is the first release of rangeland, a remote sensing pipeline for trend analysis in grasslands.
Rangeland is the first remote sensing pipeline in nf-core and, thus, deviates from most nf-core pipelines.
PR checklist
nf-core lint
).nextflow run . -profile test,docker --outdir <OUTDIR>
).docs/usage.md
is updated.docs/output.md
is updated.CHANGELOG.md
is updated.README.md
is updated (including new tool citations and authors/contributors).