Greptile Summary

This PR replaces if: always() with if: ${{ !cancelled() }} on the final aggregator job in the CI workflow. Combined with the existing cancel-in-progress: true concurrency group, this allows GitHub Actions to properly cancel the final job on superseded runs, avoiding wasted runner time and confusing failure annotations on stale SHAs. The logic of the aggregator itself is unchanged.

Confidence Score: 5/5

Safe to merge — minimal, well-motivated change with no impact on build/test logic.

Single-line change to a CI condition with a clear explanation. The !cancelled() expression is the standard GitHub Actions idiom for this pattern and correctly interacts with cancel-in-progress: true. No code paths, tests, or release logic are affected.

No files require special attention.

Important Files Changed

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[New push triggers workflow] --> B{cancel-in-progress}
    B -- "supersedes previous run" --> C[Previous run cancelled]
    C --> D{final job condition}
    D -- "always() — OLD" --> E[final runs anyway\nwastes runner, noisy annotation]
    D -- "!cancelled() — NEW" --> F[final skipped\nclean cancellation]
    B -- "no superseding push" --> G[test + msrv jobs run]
    G --> H{all success?}
    H -- yes --> I[final runs — exits 0]
    H -- no --> J[final runs — exits 1\nmarks PR red]

_{Reviews (2): Last reviewed commit: "Merge branch 'main' into ci/cancelled-fi..." | Re-trigger Greptile}