-
Notifications
You must be signed in to change notification settings - Fork 82
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
PR for release 2.0.3 #97
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Pull latest updates
…heir own lab's default references than iGenomes
Syncing upstream changes
Upstream change syncing
1) moved some previously optional but now mandatory arguments to correct section 2) added --pairedEnd message 3) renamed 'options' to be more specific (as referring to data type) 4) specified `--clip_forward_*` flag refers to sequence
Very long table of contents and inconsistent depth makes hard to find correct section quickly
Documentation improvements
Small change :-)
Updated and expanded readme for clarity and precision
Adjustments for #76 , rename certain options to be more explicit
Moved bash profile config here. This should be set by system administrators setting up nextflow on their system for everyone to use (assuming there is is high-through computing service), or a single user setting this up themselves would have to read installation anyway. Having this in usage would confuse bioinformatics novices.
Moved bash profile config to installation section. The bash variable stuff should be set by system administrators setting up nextflow on their system for everyone to use (assuming there is is a centralised computing service), and also a single user setting this up themselves would have to read installation anyway. Having this here would confuse/scare off bioinformatics novices.
Moved profile options to installation
Fixed program reference markdown screw up - bold not italics...
Some documentation shuffling to account for users vs administrators
Will get back to this once James tested successfully! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Many thanks to contributing to nf-core/eager!
Added
Fixed
Other