-
Notifications
You must be signed in to change notification settings - Fork 45
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
Docstring patches #983
Docstring patches #983
Conversation
Extra newline causes markdown to interpret them as two different lists. Remove newline to keep it all as one list.
Lists were already formatted as bullet lists with extraneous numbering or as manual plaintext lists.
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.
Thankyou @abhro for this valuable contribution 🙏🏾 . Just a couple of minor comments.
Co-authored-by: Anthony Blaom, PhD <anthony.blaom@gmail.com>
Update variable name style (and comments) to match docs
I think I've incorporated them. Thank you! |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## dev #983 +/- ##
==========================================
+ Coverage 88.13% 88.20% +0.06%
==========================================
Files 28 28
Lines 2587 2585 -2
==========================================
Hits 2280 2280
+ Misses 307 305 -2 ☔ View full report in Codecov by Sentry. |
Major changes:
For more details, please see the commit messages. Thank you!