Skip to content

Conversation

namurphy
Copy link
Member

@namurphy namurphy commented Jul 15, 2024

This PR adds functionality to docs/_global_substitutions.py to generate a table of the global substitutions that are used in the documentation. Here is a link to the RTD preview of the substitutions section which contains the table.

The table is quite long, so I put it under a collapsible tab using sphinx_collapse. The table is also quite wide because one of the substitutions includes some sentences to caution about unstable APIs, but that would take a while to fix so I'm okay with leaving it as-is.

🎨 I also expanded usage of the :rst: role so that in-line reStructuredText code snippets get formatted with pygments, which I think makes it considerably more readable.

😩 All the nested backticks reminded me of writing the documentation guide in the first place!

Closes #1550.

@github-actions github-actions bot added docs PlasmaPy Docs at http://docs.plasmapy.org contributor guide documentation infrastructure python Pull requests that update Python code labels Jul 15, 2024
@github-actions github-actions bot added the maintenance General updates to package infrastructure label Jul 15, 2024
Copy link

codecov bot commented Jul 15, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 95.23%. Comparing base (4d09035) to head (2d8bcd3).
Report is 74 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2766      +/-   ##
==========================================
+ Coverage   94.44%   95.23%   +0.79%     
==========================================
  Files         107      107              
  Lines        9597     9597              
  Branches     2221     2221              
==========================================
+ Hits         9064     9140      +76     
+ Misses        343      273      -70     
+ Partials      190      184       -6     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@@ -0,0 +1 @@
Renamed :file:`docs/_cff_to_rst.py` to :file:`docs/_author_list_from_cff.py`.
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🏷️ Mentioning in the name that this is for creating an author list should lead to more readable code.

@namurphy namurphy marked this pull request as ready for review July 16, 2024 19:32
@namurphy namurphy requested a review from a team as a code owner July 16, 2024 19:32
@namurphy namurphy requested review from ejohnson-96 and removed request for a team July 16, 2024 19:32
Copy link
Contributor

@ejohnson-96 ejohnson-96 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@namurphy
Copy link
Member Author

@ejohnson-96 — thank you for the prompt review!

@namurphy namurphy merged commit 2b799b8 into PlasmaPy:main Jul 16, 2024
@namurphy namurphy deleted the global-substitutions-table branch July 16, 2024 19:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
contributor guide docs PlasmaPy Docs at http://docs.plasmapy.org documentation infrastructure maintenance General updates to package infrastructure python Pull requests that update Python code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Automatically construct a table of substitutions from _global_substitutions.py in doc guide
2 participants