agent: @U0AJM7X8FBR API - formatter github action. • actual: formatter is only #310
agent: @U0AJM7X8FBR API - formatter github action.
• actual: formatter is only #310sweetmantech wants to merge 1 commit intotestfrom
Conversation
Adds a new `format.yml` workflow that runs `pnpm format:check` (Prettier) automatically on every pull request targeting `main` or `test`, ensuring all code changes follow standardized formatting before merge. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Important Review skippedReview was skipped due to path filters ⛔ Files ignored due to path filters (1)
CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
📝 Coding Plan
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
recoup-coding-agent
left a comment
There was a problem hiding this comment.
Code Review: API formatter GitHub Action
Summary
Adds a GitHub Actions workflow (.github/workflows/format.yml) that runs pnpm format:check on all PRs targeting main and test. Ensures code formatting is enforced automatically on every PR.
Assessment
✅ Simple, correct CI addition.
✅ Uses pinned action versions (actions/checkout@v4, actions/setup-node@v4, pnpm/action-setup@v4).
✅ Only one job, one responsibility.
🔵 Nits
- Recommend caching the pnpm store for faster CI runs:
- name: Get pnpm store directory
id: pnpm-cache
run: echo "dir=$(pnpm store path)" >> $GITHUB_OUTPUT
- uses: actions/cache@v4
with:
path: ${{ steps.pnpm-cache.outputs.dir }}
key: pnpm-${{ runner.os }}-${{ hashFiles('**/pnpm-lock.yaml') }}
restore-keys: pnpm-${{ runner.os }}-- Consider adding
--frozen-lockfiletopnpm installto catch lockfile drift in CI.
Verdict: approve
Straightforward CI improvement.
Automated PR from coding agent.
Prompt: @U0AJM7X8FBR API - formatter github action.
• actual: formatter is only run manually.
• required: formatter is run automatically in a github action on all pull requests so all code changes follow standardized formatting.