Skip to content

ruff-pre-commit is pinned at v0.17.2, the newest tag, so prek-pins-current passes again - #419

Merged
HackingGate merged 1 commit into
mainfrom
ruff-0.17.2
Oct 23, 2026
Merged

HackingGate merged 1 commit into
mainfrom
ruff-0.17.2

Conversation

@HackingGate

Copy link
Copy Markdown
Owner

Upstream tagged ruff-pre-commit v0.17.2 while #417 and #418 were in CI, and both failed only on the manual-stage prek-pins-current check, on a line neither touched. ruff check and ruff format under v0.17.2 pass over the whole tree.

Merge this first, then rebase #417 and #418.

https://claude.ai/code/session_01YTpdnDJrf2hT6hFyjS7evA

…rrent passes again

Upstream tagged v0.17.2 while #417 and #418 were in CI, and the manual-stage
pin check failed both on a line neither touched. ruff check and ruff format
under v0.17.2 report nothing new over the whole tree.

Claude-Session: https://claude.ai/code/session_01YTpdnDJrf2hT6hFyjS7evA
@coderabbitai

coderabbitai Bot commented Oct 23, 2026

Copy link
Copy Markdown

Warning

Review limit reached

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Next included review available in 39 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

Learn how review limits work.

Review configuration:

⚙️ Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 587b8fa3-1ba0-4b2a-a2bf-375dde9fc4ac
📥 Commits

Reviewing files that changed from the base of the PR and between 8f0d3d4 and 9924234.

📒 Files selected for processing: 1
  • .pre-commit-config.yaml
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@HackingGate
HackingGate merged commit d84013c into main Oct 23, 2026
12 checks passed
@HackingGate
HackingGate deleted the ruff-0.17.2 branch October 23, 2026 13:39
@codecov-commenter

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 94.85%. Comparing base (8f0d3d4) to head (9924234).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #419   +/-   ##
=======================================
  Coverage   94.85%   94.85%           
=======================================
  Files          52       52           
  Lines       27259    27259           
=======================================
  Hits        25857    25857           
  Misses       1402     1402           

☔ View full report in Codecov by Harness.
📢 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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants