Skip to content

Conversation

@pontemonti
Copy link
Contributor

Adds a Claude Code skill that enforces code quality before committing:

  • Checks formatting with ruff format
  • Runs linting with ruff check
  • Executes unit tests
  • Auto-generates commit messages

Adds a Claude Code skill that enforces code quality before committing:
- Checks formatting with ruff format
- Runs linting with ruff check
- Executes unit tests
- Auto-generates commit messages

Co-Authored-By: Claude <noreply@anthropic.com>
@pontemonti pontemonti requested a review from a team as a code owner January 22, 2026 23:23
Copilot AI review requested due to automatic review settings January 22, 2026 23:23
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds a Claude Code /commit skill that quality-gates commits by running formatting, linting, and unit tests before generating a commit message and committing staged changes.

Changes:

  • Add a new commit skill definition with step-by-step quality gates (ruff format, ruff lint, pytest).
  • Add commit-message generation guidelines and a standardized commit execution flow.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings January 22, 2026 23:28
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.

Reorders quality checks to run linting before formatting, matching
the CI pipeline order in .github/workflows/ci.yml. Adds a re-verify
step after auto-fixes to prevent commits that pass locally but fail
CI when lint fixes introduce formatting issues.

Co-Authored-By: Claude <noreply@anthropic.com>
@pontemonti pontemonti enabled auto-merge (squash) January 23, 2026 23:03
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