Skip to content

Conversation

@siliataider
Copy link
Contributor

@siliataider siliataider commented Feb 3, 2026

PR #21068 switched ruff to diff the last two commits instead of origin/<base>, but formatting still used origin/<base> (which no longer existed in the local history).
This PR aligns formatting with the new logic.

@siliataider siliataider self-assigned this Feb 3, 2026
@github-actions
Copy link

github-actions bot commented Feb 3, 2026

Test Results

0 tests   0 ✅  0s ⏱️
0 suites  0 💤
0 files    0 ❌

Results for commit 9960495.

♻️ This comment has been updated with latest results.

@siliataider siliataider force-pushed the ruff-formatting-linting branch 3 times, most recently from e7ef7b2 to 24157c7 Compare February 3, 2026 15:30
@siliataider siliataider changed the title [ci] Fix Ruff formatting [ci] Ensure origin/<base> exists for ruff formatting Feb 3, 2026
@siliataider siliataider force-pushed the ruff-formatting-linting branch 2 times, most recently from e2f44a1 to 35d9062 Compare February 3, 2026 15:45
@siliataider siliataider changed the title [ci] Ensure origin/<base> exists for ruff formatting [ci] Apply last-2-commits diff logic to ruff formatting Feb 3, 2026
@siliataider siliataider force-pushed the ruff-formatting-linting branch from 35d9062 to 9960495 Compare February 3, 2026 15:48
@siliataider siliataider marked this pull request as ready for review February 3, 2026 15:48
@siliataider siliataider requested a review from dpiparo as a code owner February 3, 2026 15:48
@siliataider siliataider requested a review from hageboeck February 3, 2026 15:51
@siliataider siliataider added the skip ci Skip the full builds on the actions runners label Feb 3, 2026
@siliataider siliataider closed this Feb 3, 2026
@siliataider siliataider reopened this Feb 3, 2026
@siliataider siliataider merged commit eec4310 into root-project:master Feb 3, 2026
23 of 45 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

in:CI skip ci Skip the full builds on the actions runners

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants