Skip to content

Add commit skill for quality-gated commits#128

Merged
pontemonti merged 5 commits intomainfrom
users/johanb/Skill
Feb 23, 2026
Merged

Add commit skill for quality-gated commits#128
pontemonti merged 5 commits intomainfrom
users/johanb/Skill

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
Copilot AI review requested due to automatic review settings February 6, 2026 17: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

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

@github-actions
Copy link

⚠️ Deprecation Warning: The deny-licenses option is deprecated for possible removal in the next major release. For more information, see issue 997.

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

None

@pontemonti pontemonti merged commit dfb34c0 into main Feb 23, 2026
9 checks passed
@pontemonti pontemonti deleted the users/johanb/Skill branch February 23, 2026 18:25
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.

4 participants