-
Notifications
You must be signed in to change notification settings - Fork 133
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
Fix some typos around backticks in docstrings #4463
Conversation
277c3a0
to
599217a
Compare
I was looking into adding this to the docstring checks, but then noticed that it should not be necessary, the doctests do run fine with the space after the backticks and even with the extra text after the For example in the logs on master we have:
(for the line with |
Good to know @benlorenz. The further words like |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #4463 +/- ##
=======================================
Coverage 84.41% 84.41%
=======================================
Files 668 668
Lines 88598 88598
=======================================
+ Hits 74786 74788 +2
+ Misses 13812 13810 -2
|
If I remember correctly, we initially used these "further words" a lot more in So this is a welcome change on our part. |
This also removes some faulty spaces between triple backticks andjldoctest
and thus enables many doctests inexperimental/AlgebraicStatistics
(cc @bkholler @taboege @marinagarrote)