Skip to content

fix(ci): format UI files to restore green main#6

Merged
DCCA merged 1 commit into
mainfrom
claude/exciting-mccarthy-l225ql
Jun 22, 2026
Merged

fix(ci): format UI files to restore green main#6
DCCA merged 1 commit into
mainfrom
claude/exciting-mccarthy-l225ql

Conversation

@DCCA

@DCCA DCCA commented Jun 22, 2026

Copy link
Copy Markdown
Owner

Fix: CI red on main

After PR #2 merged (which added a format:check CI gate), PRs #3#5 (the UI refactor) and a new CLAUDE.md landed on main without Prettier formatting, so CI on main failed at the format:check step (typecheck and lint passed; test/build never ran).

This applies Prettier to the 4 unformatted files — no logic changes:

  • CLAUDE.md
  • src/components/ui/badge.tsx
  • src/editor/main.tsx
  • src/viewer/main.tsx

Verification (all green locally)

  • npm run typecheck
  • npm run lint
  • npm run format:check
  • npm run test ✓ (34 tests)
  • npm run build

🤖 Generated with Claude Code

https://claude.ai/code/session_01PJ5c2e5VcKVhugy7TAzi4W


Generated by Claude Code

PRs #3-#5 (UI refactor) and a new CLAUDE.md landed on main without
Prettier formatting, so the format:check gate added in the hardening
sweep failed on main. Format the 4 offending files (CLAUDE.md,
badge.tsx, editor/main.tsx, viewer/main.tsx). No logic changes.

All gates pass locally: typecheck, lint, format:check, test (34), build.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01PJ5c2e5VcKVhugy7TAzi4W
@DCCA DCCA merged commit 4a6be5f into main Jun 22, 2026
2 checks passed
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