Automatic Review

The added concurrency block is generally a good practice to prevent overlapping runs of the same workflow on the same branch or PR, which improves efficiency and resource usage.

However, using ${{ github.ref }} as part of the concurrency group can cause unintended behavior because github.ref may contain slashes (refs/heads/feature/...), which are not allowed characters for concurrency group names and might lead to errors or incorrect grouping.

Suggestion: Sanitize or simplify the concurrency group identifier. A common pattern is to use the PR number (if available) or a sanitized version of the branch name.

Example fix using the PR number (pull requests trigger workflows and have github.event.pull_request.number):

concurrency:
  group: ${{ github.workflow }}-pr-${{ github.event.pull_request.number }}
  cancel-in-progress: true

If the workflow can also run on branches (not just PRs) and you want to cover both cases, you might need conditional expressions or choose one consistent identifier.

Summary:

• Using ${{ github.ref }} directly in concurrency.group can cause failures due to invalid characters.
• Replace it with a sanitized or simpler identifier such as PR number.