Skip to content

ci: Check formatting with oxfmt in lint - #1547

Merged
JoshuaKGoldberg merged 2 commits into
mainfrom
oxfmt-ci-check
Oct 20, 2026
Merged

JoshuaKGoldberg merged 2 commits into
mainfrom
oxfmt-ci-check

Conversation

@JoshuaKGoldberg

@JoshuaKGoldberg JoshuaKGoldberg commented Oct 19, 2026 •

Copy link
Copy Markdown
Member

oxfmt only ran in the pre-commit hook, so commits that skipped it (bot and web commits) landed unformatted. That means the next PR to touch those files picks up unrelated formatting churn.

This adds oxfmt --check to pnpm run lint, which CI already runs, and formats the existing files in a separate commit so the check starts clean.

JoshuaKGoldberg and others added 2 commits October 19, 2026 17:04
The pre-commit hook was the only place oxfmt ran, so commits that skipped it
landed unformatted and the next contributor to touch those files inherited the
churn. OpenSpec-generated agent files are ignored because `openspec update`
rewrites them.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Mechanical `pnpm run format` output so the new lint check starts clean.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@vercel

vercel Bot commented Oct 19, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
sentry-local Ready Ready Preview Oct 19, 2026 9:16pm UTC

Request Review

@BYK
BYK marked this pull request as ready for review October 19, 2026 21:35
@JoshuaKGoldberg
JoshuaKGoldberg merged commit 06d5a3d into main Oct 20, 2026
40 of 42 checks passed
@JoshuaKGoldberg
JoshuaKGoldberg deleted the oxfmt-ci-check branch October 20, 2026 14:38

This branch was successfully deployed

1 active and 1 inactive deployments
Preview — 2f9f92f0 Deployed Oct 19, 2026 by vercel[bot]
Actions — 2f9f92f0 Deployed Oct 19, 2026 by JoshuaKGoldberg via eval #1330
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