Skip to content

Conversation

@namurphy
Copy link
Member

📚 This PR adds a section to the coding guide within the contributor guide with the necessary steps to add a new module to PlasmaPy.

There are a few formulary-specific steps that need to be included. I'd like to add a little more discussion about adding narrative documentation (and potentially example Jupyter notebooks) too, though this may be a bit more nebulous and possibly subpackage-specific. 🤔

This PR is inspired by the contributions by @Getahun-yw-Mahl and @josephrhsmith in #3021 to add src/plasmapy/formulary/laser.py). 🌱 Many thanks!

@codecov
Copy link

codecov bot commented May 27, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 94.61%. Comparing base (5c17b8f) to head (8700af4).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #3023   +/-   ##
=======================================
  Coverage   94.61%   94.61%           
=======================================
  Files         107      107           
  Lines        9664     9664           
  Branches     1464     1464           
=======================================
  Hits         9144     9144           
  Misses        327      327           
  Partials      193      193           

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.


To get a better idea of the steps necessary to add a new module,
check out a recent pull request that adds a new module (e.g.,
:pr:`3021`).
Copy link
Member Author

Choose a reason for hiding this comment

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

Suggested change
:pr:`3021`).
:pr:`3021` or :pr:`3055`).

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant