-
-
Notifications
You must be signed in to change notification settings - Fork 52
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
chore(deps): make Qt backend optional #350
Conversation
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #350 +/- ##
==========================================
- Coverage 80.33% 79.68% -0.65%
==========================================
Files 21 22 +1
Lines 1795 1797 +2
==========================================
- Hits 1442 1432 -10
- Misses 353 365 +12 ☔ View full report in Codecov by Sentry. |
TODO: - [ ] Add relevant entry in CHANGELOG - [ ] Update install documentation - [ ] Make sure `manim-slides convert` can run without any Qt backend - [ ] Make sure test suite works (partially) without any Qt backend - [ ] Make sure we can import `manim_slides` without any Qt backend
for more information, see https://pre-commit.ci
TODO: - [ ] Add relevant entry in CHANGELOG - [ ] Update install documentation - [ ] Make sure `manim-slides convert` can run without any Qt backend - [ ] Make sure test suite works (partially) without any Qt backend - [ ] Make sure we can import `manim_slides` without any Qt backend
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
docs/source/installation.md
Outdated
|
||
As of `v5.1`, you **need** to have Qt bindings installed to use | ||
`manim-slides present` or `manim-slides wizard`. The recommended way to | ||
install those is via optional dependencies, as explained above. |
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.
[LanguageTool] reported by reviewdog 🐶
The verb ‘is’ is singular. Did you mean: “this is” or “those are”? (SINGULAR_VERB_AFTER_THESE_OR_THOSE[1])
Suggestions: this is
, those are
Rule: https://community.languagetool.org/rule/show/SINGULAR_VERB_AFTER_THESE_OR_THOSE?lang=en-US&subId=1
Category: GRAMMAR
TODO:
manim-slides convert
can run without any Qt backendmanim_slides
without any Qt backend