Skip to content

Conversation

snomiao
Copy link
Member

@snomiao snomiao commented Sep 4, 2025

Summary

  • Modified the Chromatic workflow to only publish builds when pushing to the main branch
  • PR builds will still run Storybook build for validation but skip the Chromatic publish step
  • This reduces unnecessary Chromatic usage and focuses visual testing on main branch changes

Changes

  • Added conditional if: github.event_name == 'push' && github.ref == 'refs/heads/main' to the Chromatic publish step
  • Disabled PR commenting step since we're not publishing on PRs anymore
  • Storybook build still runs for all branches to ensure it compiles correctly

Benefits

  • Reduces Chromatic build usage/costs
  • Faster PR builds (no waiting for Chromatic)
  • Visual testing focused on production (main branch) changes only

🤖 Generated with Claude Code

@vercel
Copy link

vercel bot commented Sep 4, 2025

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

Project Deployment Preview Comments Updated (UTC)
registry-web Ready Ready Preview Comment Sep 22, 2025 10:23am

@snomiao snomiao marked this pull request as ready for review September 4, 2025 16:52
@Copilot Copilot AI review requested due to automatic review settings September 4, 2025 16:52
@snomiao snomiao enabled auto-merge (squash) September 4, 2025 16:53
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR updates localization files to add new translation keys for node version management features and modifies the Chromatic workflow to optimize build usage by only publishing to Chromatic on main branch pushes.

  • Added new translation keys for bulk update functionality and version compatibility features across all supported languages
  • Modified Chromatic workflow to conditionally publish only on main branch pushes
  • Disabled PR commenting since Chromatic builds no longer run for PRs

Reviewed Changes

Copilot reviewed 9 out of 9 changed files in this pull request and generated 1 comment.

File Description
locales/*/common.json Added translations for new node version management features
.github/workflows/chromatic.yml Modified to only publish to Chromatic on main branch pushes

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

- Skip Chromatic publish step in PRs to reduce unnecessary builds
- Only run Chromatic deployment when pushing to main branch
- Keep Storybook build for all branches for validation
- Disable PR commenting since we're not publishing on PRs

This change reduces Chromatic usage and focuses visual testing on main branch changes only.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>
@snomiao snomiao force-pushed the sno-reduce-chromatic branch from 95c6355 to 2cbda14 Compare September 22, 2025 10:22
@snomiao snomiao merged commit c131ea5 into main Sep 22, 2025
8 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.

1 participant