Skip to content

feat: add pre-pr-checks skill for Claude Code#109

Open
alenkacz wants to merge 1 commit into
mainfrom
av/pre-pr-checks-skill
Open

feat: add pre-pr-checks skill for Claude Code#109
alenkacz wants to merge 1 commit into
mainfrom
av/pre-pr-checks-skill

Conversation

@alenkacz
Copy link
Copy Markdown
Contributor

Summary

  • Add .claude/skills/pre-pr-checks.md — a Claude Code skill that maps the CI pipeline to local commands
  • Documents task ci as the single command to run all checks (license, lint, unit tests)
  • Calls out common lint rules (wsl_v5, funcorder, gocritic) that frequently trip up contributors

Context

We hit wsl_v5 lint failures on #108 that could have been caught locally. This skill ensures Claude Code runs the right checks before pushing, matching the ADP pre-pr-checks pattern in cloudv2.

Test plan

  • Verified task ci runs successfully on clean main
  • Skill follows the established pattern from cloudv2's adp-pre-pr-checks skill

🤖 Generated with Claude Code

Maps the CI pipeline (license:check, lint, test:unit) to local
commands so issues are caught before pushing. Documents common lint
rules (wsl_v5, funcorder) that frequently trip up contributors.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@weeco
Copy link
Copy Markdown
Contributor

weeco commented Apr 28, 2026

@alenkacz Can we not just make it more explicit in the CLAUDE.md - which btw already mentions task ci?

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.

3 participants