Skip to content

Enable new rules for ruff check#3077

Merged
namurphy merged 4 commits intoPlasmaPy:mainfrom
namurphy:ruff-rules-20250804
Aug 5, 2025
Merged

Enable new rules for ruff check#3077
namurphy merged 4 commits intoPlasmaPy:mainfrom
namurphy:ruff-rules-20250804

Conversation

@namurphy
Copy link
Member

@namurphy namurphy commented Aug 5, 2025

This PR enables a few new rules for the ruff linter, including the FURB and Q rule sets. Running ruff check with these new rules resulted in no violations, so no changes to the code base were necessary. 🎉🎂💖🎆🪩🌌🥦

@github-actions github-actions bot added the packaging Related to packaging or distribution label Aug 5, 2025
@codecov
Copy link

codecov bot commented Aug 5, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 94.71%. Comparing base (e98b8b2) to head (bf48338).
⚠️ Report is 51 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3077      +/-   ##
==========================================
- Coverage   95.50%   94.71%   -0.80%     
==========================================
  Files         108      108              
  Lines        9869     9869              
  Branches     1501     1501              
==========================================
- Hits         9425     9347      -78     
- Misses        254      325      +71     
- Partials      190      197       +7     

☔ 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 added the no changelog entry needed For PRs that should not have a changelog entry label Aug 5, 2025
@namurphy namurphy merged commit 4031b63 into PlasmaPy:main Aug 5, 2025
26 of 27 checks passed
@namurphy namurphy deleted the ruff-rules-20250804 branch August 5, 2025 01:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no changelog entry needed For PRs that should not have a changelog entry packaging Related to packaging or distribution

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant