[THROWAWAY — DO NOT MERGE] live-fire test of ruff CI gate (PR #34) - #35
Closed
scottconverse wants to merge 1 commit into
Closed
scottconverse wants to merge 1 commit into
scottconverse wants to merge 1 commit into
Conversation
Collaborator
Author
|
Live-fire test PASSED: ruff (lint) CI job correctly failed in 41s on this PR's deliberate F401 violation, while sibling jobs (frontend, T2C) passed. New gate from PR #34 works as designed. Closing throwaway, deleting branch. Outcome logged in sprint-phase2-log.md. |
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.
Live-fire test of the ruff CI gate added in #34 (now merged at
1a19a9e). This PR contains a single deliberateF401violation (import oswith no usage inbackend/tests/test_ruff_gate_livefire_throwaway.py).Expected: ruff (lint) CI job FAILS. All other jobs (backend pytest, frontend, T2C smoke) pass since the throwaway file doesn't break runtime.
After CI confirms ruff fails: I'll close this PR without merge and delete the branch. The unmerged commit goes away with the branch.
Per dev directive: live-fire test of new gate after #33 merge. Sprint log will record the outcome.