Skip to content

Update validators.py#2919

Merged
namurphy merged 10 commits intoPlasmaPy:mainfrom
Physics-is-awesome:First_issue
Jul 31, 2025
Merged

Update validators.py#2919
namurphy merged 10 commits intoPlasmaPy:mainfrom
Physics-is-awesome:First_issue

Conversation

@Physics-is-awesome
Copy link
Contributor

This is my attempt to resolve issue #2788 by swiching every checks_on_return with validations_on_return

@github-actions github-actions bot added plasmapy.utils Related to the plasmapy.utils subpackage python Pull requests that update Python code labels Nov 25, 2024
@github-actions
Copy link

Thank you for submitting this pull request (PR)! ✨ PlasmaPy depends on contributors like you, so we deeply appreciate it!

The bottom of this page shows several checks that are run for every PR. ✅ Don't worry if something broke! We break stuff all the time. 😅 Click on "Details" to learn why a check didn't pass. Please also feel free to ask for help. We do that all the time as well! 🌸 We invite you to our weekly community meeting (2 pm ET on most Tuesdays) and office hours (3 pm ET on most Thursdays).

After this PR is marked as ready for review, someone should stop by to provide a code review and offer suggestions soon. 👥 If we don't provide a review within a few days, please feel free to send us a reminder. 🎗

PlasmaPy's contributor guide has information on:

Tip

📚 For a documentation preview, click on Details next to docs/readthedocs.org:plasmapy. For cryptic documentation errors, see the documentation troubleshooting guide.

Tip

🧹 Automatically fix most pre-commit.ci failures by commenting pre-commit.ci autofix below. For other failures, please see the pre-commit troubleshooting guide.

We thank you once again! 🌌

@namurphy namurphy self-requested a review November 26, 2024 00:51
@namurphy namurphy added the no changelog entry needed For PRs that should not have a changelog entry label Nov 26, 2024
@namurphy namurphy requested a review from a team as a code owner July 31, 2025 19:08
@github-actions github-actions bot added the maintenance General updates to package infrastructure label Jul 31, 2025
@namurphy namurphy changed the title Resolving issue #2788 Update docstrings related to @validate_quantities Jul 31, 2025
@namurphy namurphy added the needed for release Must be address/merged before next release label Jul 31, 2025
@namurphy namurphy added this to the v2025.7.0 milestone Jul 31, 2025
@namurphy namurphy changed the title Update docstrings related to @validate_quantities Update validators.py Jul 31, 2025
@codecov
Copy link

codecov bot commented Jul 31, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 95.50%. Comparing base (d41cb81) to head (4f5c6a5).
⚠️ Report is 59 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2919   +/-   ##
=======================================
  Coverage   95.50%   95.50%           
=======================================
  Files         108      108           
  Lines        9869     9869           
  Branches     1501     1501           
=======================================
  Hits         9425     9425           
  Misses        254      254           
  Partials      190      190           

☔ 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.

@namurphy namurphy removed the no changelog entry needed For PRs that should not have a changelog entry label Jul 31, 2025
@namurphy namurphy enabled auto-merge (squash) July 31, 2025 19:43
@namurphy namurphy merged commit 0e7e5cc into PlasmaPy:main Jul 31, 2025
21 of 22 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

maintenance General updates to package infrastructure needed for release Must be address/merged before next release plasmapy.utils Related to the plasmapy.utils subpackage python Pull requests that update Python code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants