-
Notifications
You must be signed in to change notification settings - Fork 16
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
✨ Test Documentation code snippets #3386
Conversation
ac343f9
to
aed58b5
Compare
aed58b5
to
25ecf71
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## develop #3386 +/- ##
===========================================
+ Coverage 74.51% 74.61% +0.10%
===========================================
Files 230 230
Lines 26813 26812 -1
===========================================
+ Hits 19979 20007 +28
+ Misses 6834 6805 -29 ☔ View full report in Codecov by Sentry. |
e9db1e7
to
d11df0e
Compare
d11df0e
to
3cb0e83
Compare
|
4e45a8b
to
85be89b
Compare
Quality Gate passedIssues Measures |
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.
Great feature and getting the docs more up to scratch!
Description
This is a possible addition to our CI suite that tests all code snippets in documentation and docstrings.
The documentation tests cant be run in parallel currently because each file shares a namespace.
The choice of using Sybil needs to be discussed but so far looks promising
Interface Changes
Checklist
I confirm that I have completed the following checks:
pytest tests --reactor
pre-commit run --from-ref develop --to-ref HEAD
sphinx-build -W documentation/source documentation/build