-
Notifications
You must be signed in to change notification settings - Fork 78
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
[MRG] Add sourmash logo to docs and README #1127
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1127 +/- ##
==========================================
+ Coverage 83.79% 93.04% +9.25%
==========================================
Files 98 74 -24
Lines 9063 5810 -3253
==========================================
- Hits 7594 5406 -2188
+ Misses 1469 404 -1065
Flags with carried forward coverage won't be shown. Click here to find out more. Continue to review full report at Codecov.
|
👍 |
Ready for review and merge @ctb @taylorreiter |
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.
go for it!
Fixes #673
I also removed the customized template for the docs, opting for using the one that comes by default with Alabaster. It is a bit different (for example, "Navigation" instead of "Table of Contents" in the sidebar), but this way we don't need to keep updating our copy when the one from the theme is updated.
TODO:
Checklist
make test
Did it pass the tests?make coverage
Is the new code covered?without a major version increment. Changing file formats also requires a
major version number increment.
changes were made?