-
Notifications
You must be signed in to change notification settings - Fork 13
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 Snakemake style guide #162
Conversation
To something less general than "style" so other style guides can be added without confusion.
Copied from the internal wiki¹. ¹ https://nextstrain.atlassian.net/wiki/pages/viewpage.action?pageId=170459138 Co-authored-by: John Huddleston <huddlej@gmail.com>
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.
Looks great, thanks!
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.
+1 for this!
Let's also deprecate/archive/delete https://github.com/blab/styleguide while we're at it? Once your PR is merged, we can:
- update the nextstrain-builds.md document to link to the page added by your PR: https://docs.nextstrain.org/page/reference/snakemake-style-guide.html
- update the README to say this is unused/never used, and archive the repo (so old links still work)
and then that old repo/guide will finally be cleaned up.
I do think there's a couple things that would be worth porting to the new style guide (the use of :q
really should be a no-brainer in our workflows, alas it's not yet), but we can do that in a separate PR.
This is consistent with the existing documentation style guide and a similar Snakemake style guide¹ written by @tsibley. This also allows for the addition of a table of contents, which is done here. I looked into making the section links future-proof to wording changes, however could not find a good solution. The closest is using :ref: roles which adds separately defined links to to the section titles, but those links aren't used by the TOC or header links themselves. ¹ https://github.com/blab/styleguide/blob/6aa5d7aa42acfa97e57a5ee05a4175f158502cac/nextstrain-builds.md
preview of new page
Description of proposed changes
Publish the internal Snakemake style guide as a page on the public docs.
Related issue(s)
This was discussed on Slack [1] [2].
Testing