NO-SNOW: Add repo-local graphite-pr-workflow skill with mvn format + checkstyle pre-commit rules#2598
Conversation
This stack of pull requests is managed by Graphite. Learn more about stacking. |
There was a problem hiding this comment.
Pull request overview
Adds a repo-local Cursor skill to standardize how contributors use Graphite CLI for creating branches/commits/PRs in snowflake-jdbc, including explicit “don’t commit/submit without instruction” guidance and required pre-commit Maven formatting/checkstyle steps.
Changes:
- Introduces a new
.cursorskill documenting the Graphite PR workflow for this repo. - Documents required pre-commit commands (Spotify fmt +
check-styleprofile validation). - Provides commit message conventions and a PR description template.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Merge activity
|
…checkstyle pre-commit rules Made-with: Cursor
sfc-gh-dheyman
force-pushed
the
dheyman/add-graphite-skill
branch
from
fc5a8ac to
8ff704b
Compare
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 1 out of 1 changed files in this pull request and generated 4 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Adds a Cursor skill at
.cursor/skills/graphite-pr-workflow/SKILL.mdthat encodes the standard PR workflow for the snowflake-jdbc repository. The skill instructs the AI assistant to require explicit user approval before committing or submitting a PR, run the Spotify fmt plugin and checkstyle validation before any commit, follow theSNOW-{ticket}:commit message convention, prompt for a SNOW ticket number if one is not provided, detect stacked PR scenarios and usegt submit --stackaccordingly, and use the standard PR description template with Summary, Context, and Test plan sections.