Skip to content

Comments

Exit with non-zero when autofix not applied#552

Open
soraxas wants to merge 1 commit intoInstagram:mainfrom
soraxas:fix/use-non-zero-exit-code
Open

Exit with non-zero when autofix not applied#552
soraxas wants to merge 1 commit intoInstagram:mainfrom
soraxas:fix/use-non-zero-exit-code

Conversation

@soraxas
Copy link

@soraxas soraxas commented Feb 23, 2026

Summary

Closes #409

This PR improves the exit code handling in the fix command when multiple issues are encountered in a single run.

Test Plan

  • Added a new test method test_fix_exit_codes to verify the exit codes for unfixable violations, syntax errors, and scenarios where both occur together.

Signed-off-by: Tin Lai <tin@tinyiu.com>
@soraxas soraxas requested a review from amyreese as a code owner February 23, 2026 14:28
@meta-cla meta-cla bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Feb 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.

Projects

None yet

1 participant