-
Notifications
You must be signed in to change notification settings - Fork 339
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
Improve consistency and ReST formatting in documentation #1073
Improve consistency and ReST formatting in documentation #1073
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #1073 +/- ##
==========================================
- Coverage 96.95% 96.95% -0.01%
==========================================
Files 71 71
Lines 7027 7026 -1
==========================================
- Hits 6813 6812 -1
Misses 214 214 ☔ View full report in Codecov by Sentry. |
I started to implement `ParticleLike` in some docstrings, and decided that went outside the scope of this pull request.
This PR includes an update to the section in the docs on required packages, so it'd be helpful (but not required) for this to be merged before the 0.6.0 release. |
@@ -348,7 +342,7 @@ def unit(self): | |||
|
|||
Raises | |||
------ | |||
ValueError | |||
`ValueError` |
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.
I'm totally okay with this. Just pointing out that it makes no difference when rendering the RTD docs. For some reason, the numpydoc
"Raises" section never applies the code syntax when this is done.
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.
In the grids
module, all instances of DataArray
should be replaced with ~xarray.DataArray
so it's properly cross-referenced. This will also require adding xarray
to the intersphinx_mapping
dictionary in the conf.py
.
Co-authored-by: Erik Everson <[email protected]>
Co-authored-by: Erik Everson <[email protected]>
Co-authored-by: Erik Everson <[email protected]>
…lasmaPy into doc-fixes-the-next-generation
…the-next-generation
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.
LGTM
Thanks! The most recent set of changes are expanding the links (mostly |
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.
Still looks good. :)
Some minor touch-ups and updates to the documentation, including to the stability matrix. The changes should be pretty straightforward.