ci: stop auto-accepting Chromatic changes on main - #186
Merged
Conversation
`--auto-accept-changes main` took whatever landed as the new baseline, which is how a broken shadcn mapping became the reference in #184 with no one looking. Changes get accepted in the PR review, where there is a diff to look at. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01FtANJj1cbBZpj12mEXjgSN
abernier
force-pushed
the
fix/tailwind-vite-plugin
branch
from
August 18, 2026 11:10
b3e1a7a to
b82df77
Compare
abernier
changed the title
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
--auto-accept-changes maintook whatever landed as the new baseline. That is how a broken shadcn mapping became the reference in #184 with no one looking — the PR build did show it (24 changes), but among 24 legitimate ones from the scaffold refresh, and the merge tomainthen locked it in.Changes now get accepted in the PR review, where there is a diff to look at.
Note the trade-off: a push straight to
mainwill leave the UI Tests check red until someone accepts.🤖 Generated with Claude Code
https://claude.ai/code/session_01FtANJj1cbBZpj12mEXjgSN