taxprofiler / taxpasta

TAXnomic Profile Aggregation and STAndardisation
https://taxpasta.readthedocs.io/
Apache License 2.0
34 stars 7 forks source link

Document optional dependencies #70

Closed Midnighter closed 1 year ago

Midnighter commented 1 year ago
jfy133 commented 1 year ago

Does the list of extras for development include the extra packages needed for rendering the website?

Midnighter commented 1 year ago

Does the list of extras for development include the extra packages needed for rendering the website?

Good point, they don't. Those are separately in docs/requirements.txt. Let's see how we can do this. 🤔

codecov-commenter commented 1 year ago

Codecov Report

Patch and project coverage have no change.

Comparison is base (c557970) 82.10% compared to head (33debbb) 82.10%.

:exclamation: Current head 33debbb differs from pull request most recent head 1973838. Consider uploading reports for the commit 1973838 to get more accurate results

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

Additional details and impacted files ```diff @@ Coverage Diff @@ ## dev #70 +/- ## ======================================== Coverage 82.10% 82.10% ======================================== Files 104 104 Lines 1576 1576 Branches 279 133 -146 ======================================== Hits 1294 1294 Misses 241 241 Partials 41 41 ``` Help us with your feedback. Take ten seconds to tell us [how you rate us](https://about.codecov.io/nps?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=None). Have a feature suggestion? [Share it here.](https://app.codecov.io/gh/feedback/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=None)

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

Midnighter commented 1 year ago

I've added another extra docs for those dependencies. So at least it's within the same system but for now separate from the pure dev dependencies. I did this only because historically I've been keeping them apart. They could easily be combined, however.