Skip to content

Add concurrency control and ruff configuration improvements#3

Merged
jooh merged 1 commit intomainfrom
claude/fix-ruff-templated-repos-dP2Gw
Feb 6, 2026
Merged

Add concurrency control and ruff configuration improvements#3
jooh merged 1 commit intomainfrom
claude/fix-ruff-templated-repos-dP2Gw

Conversation

@jooh
Copy link
Copy Markdown
Owner

@jooh jooh commented Feb 6, 2026

Summary

This PR improves the CI/CD workflow and code quality tooling configuration by adding concurrency controls to prevent duplicate workflow runs and configuring Ruff to exclude lock files.

Key Changes

  • GitHub Actions Workflow: Added concurrency configuration to the ruff-autofix workflow to cancel in-progress runs when a new push occurs on the same branch, reducing unnecessary CI resource usage
  • Ruff Configuration: Added exclusion rule in pyproject.toml to prevent Ruff from linting the uv.lock file, which is a generated dependency lock file that shouldn't be analyzed

Implementation Details

  • The concurrency group is keyed by github.ref to ensure each branch has its own concurrency context
  • The cancel-in-progress: true setting ensures that older workflow runs are cancelled when a newer commit is pushed to the same branch
  • The uv.lock exclusion prevents unnecessary linting of auto-generated lock file content

https://claude.ai/code/session_01YFmaXoosHkGyN3uEF3k54F

….lock

Add concurrency group to ruff-autofix workflow so that when multiple
commits are pushed (e.g. PR merge), only the latest run proceeds and
stale runs are cancelled. This prevents push failures when the workflow
runs on a commit that is no longer the branch HEAD.

Also add uv.lock to ruff's exclude list in pyproject.toml since uv
generates lock files that trigger ruff violations.

https://claude.ai/code/session_01YFmaXoosHkGyN3uEF3k54F
@claude
Copy link
Copy Markdown

claude Bot commented Feb 6, 2026

Code review

No issues found. Checked for bugs and CLAUDE.md compliance.

@jooh jooh merged commit de340d4 into main Feb 6, 2026
2 checks passed
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.

2 participants