-
Notifications
You must be signed in to change notification settings - Fork 193
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
add newline before table 🤞🏻 #2128
Conversation
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.
The fact that prettier is now formatting the table whitespace for you is a good sign! 👍🏻
README.md
Outdated
| shell | shell config file | command | | ||
| ----- | --------------------------------------- | -------------------------------------------------- | | ||
| bash | ~/.bashrc | `eval "$(_NF_CORE_COMPLETE=bash_source nf-core)"` | | ||
| Zsh | ~/.zshrc | `eval "$(_NF_CORE_COMPLETE=zsh_source nf-core)"` | |
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.
Lower case?
| Zsh | ~/.zshrc | `eval "$(_NF_CORE_COMPLETE=zsh_source nf-core)"` | | |
| zsh | ~/.zshrc | `eval "$(_NF_CORE_COMPLETE=zsh_source nf-core)"` | |
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.
the authors don't seem to be consistent about it,think I prefer lower case, but I was following how Wikipedia writes it 🤷🏻
Co-authored-by: Phil Ewels <phil.ewels@scilifelab.se>
@nf-core-bot fix linting |
Codecov Report
@@ Coverage Diff @@
## dev #2128 +/- ##
=======================================
Coverage 71.41% 71.41%
=======================================
Files 77 77
Lines 8358 8358
=======================================
Hits 5969 5969
Misses 2389 2389 Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
Fixes #2127 (hopefully)