Repository navigation
ci: Run validation jobs in parallel - #2431
Closed
Tim Flannagan (timflannagan) wants to merge 3 commits into
Closed
Tim Flannagan (timflannagan) wants to merge 3 commits into
Tim Flannagan (timflannagan) wants to merge 3 commits into
Conversation
Run repository verification and unit tests as independent parallel jobs so cheap formatting, code generation, and license failures report promptly. Keep run-tests as an aggregate job over both results to preserve the existing required status check without a ruleset migration. Signed-off-by: timflannagan <timflannagan@gmail.com>
Give linting its own Make-backed CI job so it runs in parallel with unit tests and non-lint verification. Keep the comprehensive make verify command by composing the focused lint and verification targets. Include all root and plugin linters behind make lint, and keep the required run-tests aggregate responsible for every validation result. Signed-off-by: timflannagan <timflannagan@gmail.com>
Tim Flannagan (timflannagan)
marked this pull request as draft
October 9, 2026 20:49
Classify formatting and static analysis as linting instead of repository verification. This puts gofmt, proto formatting, ShellCheck, go vet, and the Go linters behind the standalone lint target. Keep verification focused on generated artifacts, module state, licenses, migrations, metrics, and other repository invariants. Signed-off-by: timflannagan <timflannagan@gmail.com>
Contributor
Author
|
More to unwind, moving to draft while I play around with this more locally |
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.
Run unit tests, repository verification, and linting as separate parallel jobs
so inexpensive failures report without waiting for the test suite.
The workflow uses focused Make targets for linting and non-lint verification,
while
make verifyremains the comprehensive local entrypoint. The existingrequired
run-testscheck aggregates all three results, so branch protectionand merge-queue behavior remain unchanged.