Skip to content

Fix verify job failure from formatting drift in attachment and sheet registry sources - #54

Merged
vinutv merged 2 commits into
mainfrom
copilot/fix-verify-job-failure-again
Aug 31, 2026
Merged

Fix verify job failure from formatting drift in attachment and sheet registry sources#54
vinutv merged 2 commits into
mainfrom
copilot/fix-verify-job-failure-again

Conversation

Copilot AI commented Aug 31, 2026

Copy link
Copy Markdown
Contributor

The verify workflow was failing in the formatting check on attachment and sheet. In this repo, formatting changes to registry source files also require regenerating the published registry JSON to keep registry:check in sync.

  • Root cause

    • Applied Prettier-normalized class ordering in:
      • src/components/ui/attachment.tsx
      • src/components/ui/sheet.tsx
  • Registry sync

    • Regenerated the derived artifacts updated by those source changes:
      • public/registry/care-ui/attachment/attachment.json
      • public/registry/care-ui/sheet/sheet.json
  • Change shape

    • No behavior or API changes
    • Source and generated files now agree on the formatted component content
className={cn(
  "min-h-0 flex-1 overflow-y-auto overscroll-contain py-4 [scrollbar-gutter:stable]",
  className
)}

Co-authored-by: vinutv <16238306+vinutv@users.noreply.github.com>
@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Aug 31, 2026

Copy link
Copy Markdown

Deploying careui with  Cloudflare Pages  Cloudflare Pages

Latest commit: 3eac1eb
Status: ✅  Deploy successful!
Preview URL: https://5c2dd124.careui.pages.dev
Branch Preview URL: https://copilot-fix-verify-job-failu-rbjf.careui.pages.dev

View logs

Copilot AI changed the title [WIP] Fix the failing GitHub Actions job verify Fix verify job failure from formatting drift in attachment and sheet registry sources Aug 31, 2026
Copilot AI requested a review from vinutv August 31, 2026 11:36
@vinutv
vinutv marked this pull request as ready for review August 31, 2026 15:25
@vinutv
vinutv merged commit c5a5ba3 into main Aug 31, 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