ci: run autofix before main CI on pull requests#38394
Open
ojasarora77 wants to merge 1 commit into
Open
Conversation
Call autofix.ci as the first job in main-ci.yml for pull_request events so formatting fixes land before tests and style checks run. Remove the standalone pull_request trigger from autofix.yml to avoid parallel runs.
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.
Fixes #38291
Summary
Runs
autofix.cias the first job inmain-ci.ymlfor pull requests, so downstream tests and style checks run against autofixed code instead of racing with a separate autofix workflow.run-autofixjob tomain-ci.ymlthat callsautofix.ymlviaworkflow_callpre_job(and the rest of the pipeline) on autofix completingpull_requesttrigger fromautofix.ymlto prevent parallel runspushandmerge_grouptriggers onautofix.ymlunchangedThis uses a reusable workflow call rather than
workflow_runso child workflows continue to check out the PR head normally without extra ref plumbing.From Cursor
Test plan