Repository navigation
ci: drop Homeboy review audit and refactor lint - #3603
Merged
Merged
Conversation
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
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.
Slice of Extra-Chill/homeboy#15774.
Homeboy is deleting its code audit engine (
review audit), therefactorcommand, and the findings-to-issue reconciler that powered homeboy-action'sauto-issue/expected-commandsinputs. Consumers run the latest Homeboy, so they must stop calling these before the Homeboy deletion lands.Changes:
homeboy.yml: commands becomereview lint(dropreview auditand the PR-onlyrefactor lint). Therefactorcommand is being deleted; lint autofix stays available locally ashomeboy review lint data-machine --fix. CI never committed the refactor output, so this removes no gate.expected-commandsfrom both reusable-workflow calls (homeboy.yml,homeboy-test.yml). It only fed the auto-issue reconciler and homeboy-action is removing the input; passing an undeclared input to a reusable workflow is a startup failure, so this must merge before the action change.homeboy.json: remove the one-shottransformsentry (refactor transformconfig); the transform engine is being deleted. The lint baseline is untouched.Verification: workflow/config diff only;
homeboy.jsonparses as JSON; this PR's own CI exercises the updated commands.AI assistance: implemented by Claude (Anthropic) via OpenCode, directed and reviewed by @chubes4.