Skip to content

Comments

Matrix lint workflow by hook stage#2842

Merged
adamtheturtle merged 1 commit intomainfrom
adamtheturtle/lint-matrix-hook-stage
Feb 15, 2026
Merged

Matrix lint workflow by hook stage#2842
adamtheturtle merged 1 commit intomainfrom
adamtheturtle/lint-matrix-hook-stage

Conversation

@adamtheturtle
Copy link
Member

@adamtheturtle adamtheturtle commented Feb 15, 2026

Refactor the lint workflow to parallelize hook stage execution by adding hook-stage to the build matrix. This runs all three hook stages (pre-commit, pre-push, manual) in parallel instead of sequentially.


Note

Low Risk
Workflow-only refactor that changes CI job fan-out and log grouping but not the lint commands themselves; primary risk is increased CI load or matrix misconfiguration.

Overview
Parallelizes CI linting by hook stage. The lint.yml workflow adds hook-stage (pre-commit, pre-push, manual) to the build matrix so each stage runs as its own matrix job across both Ubuntu and Windows.

The Lint step is simplified from three sequential prek run invocations to a single command parameterized by ${{ matrix.hook-stage }}, improving visibility and reducing total wall-clock time.

Written by Cursor Bugbot for commit 74763f3. This will update automatically on new commits. Configure here.

@adamtheturtle adamtheturtle merged commit c83ee84 into main Feb 15, 2026
16 checks passed
@adamtheturtle adamtheturtle deleted the adamtheturtle/lint-matrix-hook-stage branch February 15, 2026 08:30
Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.

matrix:
python-version: ['3.13']
platform: [ubuntu-latest, windows-latest]
hook-stage: [pre-commit, pre-push, manual]
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lite-action in matrix jobs pushes incomplete auto-fixes

Medium Severity

Adding hook-stage to the matrix means pre-commit-ci/lite-action now runs in 6 parallel jobs, each containing auto-fixes from only a single hook stage. Previously, each job ran all three stages sequentially, so the action had the complete set of fixes. The pre-commit-ci/lite-action documentation requires it to be in a job that is "either by itself in the workflow or is the last job to finish," which 6 parallel matrix jobs violate. Only the last job's partial fixes will be pushed, losing fixes from the other two stages.

Additional Locations (1)

Fix in Cursor Fix in Web

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant