chore: move Ruff excludes to ruff.extend-exclude#5112
chore: move Ruff excludes to ruff.extend-exclude#5112wanghan-iapcm merged 1 commit intodeepmodeling:develfrom
ruff.extend-exclude#5112Conversation
Previously, it was in `ruff.lint.exclude`, so exclusions were not applied to `ruff format`.
There was a problem hiding this comment.
Pull request overview
This PR moves the Ruff exclusion configuration from ruff.lint.exclude to ruff.extend-exclude to ensure that exclusions apply to both linting and formatting operations.
Key Changes:
- Added a new
[tool.ruff]section withextend-excludeconfiguration - Removed the
excludeconfiguration from[tool.ruff.lint]section
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
📝 WalkthroughWalkthroughThe pull request modifies the Ruff configuration in Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Pre-merge checks and finishing touches✅ Passed checks (3 passed)
✨ Finishing touches🧪 Generate unit tests (beta)
📜 Recent review detailsConfiguration used: Repository UI Review profile: CHILL Plan: Pro 📒 Files selected for processing (1)
🧰 Additional context used🧠 Learnings (3)📓 Common learnings📚 Learning: 2025-12-12T13:40:14.334ZApplied to files:
📚 Learning: 2025-12-12T13:40:14.334ZApplied to files:
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (28)
🔇 Additional comments (2)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## devel #5112 +/- ##
=======================================
Coverage 82.15% 82.15%
=======================================
Files 709 709
Lines 72470 72469 -1
Branches 3615 3615
=======================================
Hits 59536 59536
+ Misses 11771 11770 -1
Partials 1163 1163 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
0e883dc
Previously, it was in
ruff.lint.exclude, so exclusions were not applied toruff format.Summary by CodeRabbit
✏️ Tip: You can customize this high-level summary in your review settings.