Visitar URL original
[pre-commit.ci] pre-commit autoupdate by pre-commit-ci[bot] · Pull Request #3050 · PlasmaPy/PlasmaPy · GitHub
Skip to content

[pre-commit.ci] pre-commit autoupdate - #3050

Merged
namurphy merged 19 commits into
mainfrom
pre-commit-ci-update-config
Jul 16, 2025
Merged

[pre-commit.ci] pre-commit autoupdate#3050
namurphy merged 19 commits into
mainfrom
pre-commit-ci-update-config

Conversation

@pre-commit-ci

@pre-commit-ci pre-commit-ci Bot commented Jul 7, 2025

Copy link
Copy Markdown
Contributor

@pre-commit-ci
[pre-commit.ci] pre-commit autoupdate
972b5bd 
updates:
- [github.com/python-jsonschema/check-jsonschema: 0.33.0 → 0.33.2](python-jsonschema/check-jsonschema@0.33.0...0.33.2)
- [github.com/macisamuele/language-formatters-pre-commit-hooks: v2.14.0 → v2.15.0](macisamuele/language-formatters-pre-commit-hooks@v2.14.0...v2.15.0)
- [github.com/mattlqx/pre-commit-search-and-replace: v1.0.5 → v1.1.8](mattlqx/pre-commit-search-and-replace@v1.0.5...v1.1.8)
- [github.com/crate-ci/typos: v1.33.1 → v1](crate-ci/typos@v1.33.1...v1)
- [github.com/astral-sh/ruff-pre-commit: v0.11.12 → v0.12.2](astral-sh/ruff-pre-commit@v0.11.12...v0.12.2)
@pre-commit-ci
pre-commit-ci Bot requested a review from namurphy as a code owner July 7, 2025 18:14
@github-actions github-actions Bot added CI Related to continuous integration linters Code linters and autoformatters labels Jul 7, 2025
@github-actions

github-actions Bot commented Jul 7, 2025

Copy link
Copy Markdown

Thank you for submitting this pull request (PR)! ✨

Below are checks that are being run for this PR. Click on the name of a check to learn why it didn't pass. ✅ Don't worry if something broke! We break stuff all the time. 😅

PlasmaPy's contributor guide has pages on:

Tip

📚 For a documentation preview, click on docs/readthedocs.org:plasmapy below. 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! 🌌

@github-actions github-actions Bot added the no changelog entry needed For PRs that should not have a changelog entry label Jul 7, 2025
namurphy
namurphy reviewed Jul 7, 2025
View reviewed changes
Comment thread .pre-commit-config.yaml Resolved
namurphy
namurphy reviewed Jul 7, 2025
View reviewed changes
Comment thread .pre-commit-config.yaml Resolved
@codecov

codecov Bot commented Jul 7, 2025 •
edited
Loading

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 73.33333% with 4 lines in your changes missing coverage. Please review.
✅ Project coverage is 94.70%. Comparing base (34d64c7) to head (19c3d57).
⚠️ Report is 54 commits behind head on main.

Files with missing lines Patch % Lines
src/plasmapy/particles/ionization_state.py 40.00% 2 Missing and 1 partial ⚠️
src/plasmapy/formulary/relativity.py 50.00% 1 Missing ⚠️
Additional details and impacted files 
@@            Coverage Diff             @@
##             main    #3050      +/-   ##
==========================================
+ Coverage   94.66%   94.70%   +0.04%     
==========================================
  Files         108      108              
  Lines        9738     9738              
  Branches     1473     1473              
==========================================
+ Hits         9218     9222       +4     
+ Misses        327      323       -4     
  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.

namurphy
namurphy reviewed Jul 7, 2025
View reviewed changes
Comment thread .pre-commit-config.yaml
ci:
autofix_prs: false
autoupdate_schedule: monthly
autoupdate_schedule: quarterly

@namurphy namurphy Jul 7, 2025

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I'm reducing the cadence of pre-commit autoupdates because each time I've had to revert two of the hooks.

@namurphy
namurphy requested review from a team, pheuer and rocco8773 as code owners July 7, 2025 20:11
@github-actions github-actions Bot added plasmapy.particles Related to the plasmapy.particles subpackage plasmapy.plasma Related to the plasmapy.plasma subpackage plasmapy.formulary Related to the plasmapy.formulary subpackage testing plasmapy.diagnostics Related to the plasmapy.diagnostics subpackage plasmapy.analysis Related to the plasmapy.analysis subpackage python Pull requests that update Python code labels Jul 7, 2025
@namurphy

namurphy commented Jul 7, 2025

Copy link
Copy Markdown
Member

There were a few new ruff rules that got activated in this update, so I made a bunch of fixes. I need to look back at this carefully to make sure that my changes won't have any unintended effects.

For example, ruff is now set to warn when a with pytest.warns(...) block has more than a single line in it. In some past PRs, we've had contributors add with pytest.warns(...) blocks to ignore errors. In retrospect, it would have been better to use @pytest.mark.filterwarnings. However, I need to make sure that the warnings are being tested elsewhere.

@github-actions github-actions Bot added the packaging Related to packaging or distribution label Jul 15, 2025
@namurphy namurphy mentioned this pull request Jul 15, 2025
Merged
@github-actions github-actions Bot added the requirements Related to updating requirements label Jul 15, 2025
@namurphy
namurphy enabled auto-merge (squash) July 15, 2025 23:59
@namurphy
namurphy merged commit e174a05 into main Jul 16, 2025
@github-actions github-actions Bot removed the no changelog entry needed For PRs that should not have a changelog entry label Jul 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@namurphy namurphy namurphy approved these changes
@pheuer pheuer Awaiting requested review from pheuer pheuer is a code owner
@rocco8773 rocco8773 Awaiting requested review from rocco8773 rocco8773 is a code owner

Assignees

No one assigned

Labels

CI Related to continuous integration linters Code linters and autoformatters packaging Related to packaging or distribution plasmapy.analysis Related to the plasmapy.analysis subpackage plasmapy.diagnostics Related to the plasmapy.diagnostics subpackage plasmapy.formulary Related to the plasmapy.formulary subpackage plasmapy.particles Related to the plasmapy.particles subpackage plasmapy.plasma Related to the plasmapy.plasma subpackage python Pull requests that update Python code requirements Related to updating requirements testing

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@namurphy