Skip to content

ci: test.yaml is missing the concurrency cancel-in-progress block other repos have #54

Description

@balbasty

What

Found as a side effect of fastfields-lib#52 (ccache): rapid successive test pushes on the same branch raced and overlapped, colliding on the ccache-save step, because .github/workflows/test.yaml in this repo lacks the

concurrency:
group: ${{ github.workflow }}-${{ github.ref }}cancel-in-progress: true

block that other C++/CUDA repos in the org already carry (e.g. fastfields-cpu-lib's lint.yaml, and the pattern used org-wide for the Python reusable workflows as of fastfields/.github#6).

Fix

Add the same concurrency block to this repo's test.yaml (and build.yaml/whatever else runs per-push here, if it doesn't already have one) so a newer push cancels a superseded in-flight run instead of letting them race.

Left out of fastfields-lib#52 deliberately to keep that PR's diff scoped to ccache only.

Workstream: claude-jitfields-to-fastfields

Activity

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

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions