fix(ci): clear ruff E702/E703 jamming the merge backlog - #77
Closed
4444J99 wants to merge 1 commit into
Closed
Conversation
…rge backlog The sync-reclaim organ landed via direct heal commit f9325a8 (bypassing PR CI) carrying 4 stray-semicolon violations (E702/E703) in cli/tests/test_sync_reclaim.py. That made `ruff check cli/src cli/tests web/api mcp` — the first step of the `python` CI job — fail on main, so EVERY PR branched from main inherits a red `python` check (#75, #73, #57, #56, #54, #53, #52, #51 …). The merge organ holds CI-red PRs, so the whole recent merge backlog — including the VLTIMA organ engine (#73) — was jammed. Split the compound/trailing-semicolon statements onto separate lines (pure formatting, semantics unchanged). ruff now passes; the 7 sync-reclaim tests still pass. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
This was referenced Jun 23, 2026
Owner
Author
|
Superseded by #78 (merged as c9180db), which landed the identical ruff E702/E703 split in cli/tests/test_sync_reclaim.py together with the web validator fix in one PR. origin/main is already ruff-clean (verified), so this diff is now a no-op. Closing to clear the merge backlog — reopen if ever needed. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
Removes 4 stray-semicolon ruff violations (E702/E703) in
cli/tests/test_sync_reclaim.py(lines ~178, 181, 183, 197). Pure formatting — compounda; band trailing;split onto separate lines, semantics unchanged.Why this is the keystone
The sync-reclaim organ landed via the direct
heal:commitf9325a8(bypassing PR CI), carrying these violations ontomain. ThepythonCI job runsruff check cli/src cli/tests web/api mcpfirst, so it fails onmain— and every PR branched frommaininherits a redpythoncheck. Sampled stuck PRs: #75, #73, #57, #56, #54, #53, #52, #51. The merge organ holds CI-red PRs, so the whole recent merge backlog — including the VLTIMA organ engine (#73) — is jammed behind this one lint error.Merging this green-on-its-own-tree PR first turns
mainruff-green and unblocks the rest.Verification
python3 -m ruff check cli/src cli/tests web/api mcp→ All checks passed!pytest cli/tests/test_sync_reclaim.py→ 7 passed🤖 Generated with Claude Code