Skip to content

fix(config): 🐛 merge chromatic and user-flow into react() test param - #262

Merged
iamnewton merged 4 commits into
mainfrom
fix/use-react-preset-test-param
Aug 31, 2026
Merged

fix(config): 🐛 merge chromatic and user-flow into react() test param#262
iamnewton merged 4 commits into
mainfrom
fix/use-react-preset-test-param

Conversation

@iamnewton

Copy link
Copy Markdown
Contributor

Summary

  • Removes the separate test workflow entry from workflows — it caused holocron sync to see two test entries and use last-entry-wins, stripping run-unit: false, run-storybook: true, and run-interaction: true from the generated test.yml
  • Moves run-chromatic and adds run-user-flow: true into react({ test: {...} }) so all options merge into one entry
  • Restructures config to use preset variable consistently (...preset, ...preset.repo, ...preset.workflows, ...preset.providers) instead of destructuring

Depends on

Test plan

🤖 Generated with Claude Code

Avoids a second test workflow entry that previously caused the sync tool
to use last-entry-wins and drop run-unit: false, run-storybook: true, and
run-interaction: true.  Merging all overrides into the single react() call
ensures holocron sync always generates one complete test.yml.

Depends on theholocron/configs#419 for react() to accept the test param.

Signed-off-by: Newton <5769156+iamnewton@users.noreply.github.com>
@iamnewton
iamnewton requested a review from a team as a code owner August 31, 2026 04:21
@github-actions github-actions Bot added the bug Something isn't working label Aug 31, 2026
@chromatic-com

chromatic-com Bot commented Aug 31, 2026

Copy link
Copy Markdown

Tip

All tests passed and all changes approved!

🟢 UI Tests: 15 tests unchanged
🟢 UI Review: 15 stories published -- no changes
Storybook icon Storybook Publish: 15 stories published

@cypress

cypress Bot commented Aug 31, 2026

Copy link
Copy Markdown

react    Run #239

Run Properties:  status check passed Passed #239  •  git commit 9cfffaf25e ℹ️: Merge f530bc9feaa81fd8ce1dee1cb4a084a0b4482d13 into 663b17f9c38e543a1201182d1692...
Project react
Branch Review fix/use-react-preset-test-param
Run status status check passed Passed #239
Run duration 01m 04s
Commit git commit 9cfffaf25e ℹ️: Merge f530bc9feaa81fd8ce1dee1cb4a084a0b4482d13 into 663b17f9c38e543a1201182d1692...
Committer Newton
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 0
Tests that did not run due to a developer annotating a test with .skip  Pending 0
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 1
View all changes introduced in this branch ↗︎

Signed-off-by: super-linterbot[bot] <310376783+super-linterbot[bot]@users.noreply.github.com>
@codecov

codecov Bot commented Aug 31, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #262   +/-   ##
=======================================
  Coverage   95.40%   95.40%           
=======================================
  Files           6        6           
  Lines          87       87           
  Branches       11       11           
=======================================
  Hits           83       83           
  Misses          2        2           
  Partials        2        2           
🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Picks up the test param on react() from configs#419.

Signed-off-by: Newton <5769156+iamnewton@users.noreply.github.com>
@iamnewton
iamnewton merged commit 9a5c8cf into main Aug 31, 2026
42 checks passed
@iamnewton
iamnewton deleted the fix/use-react-preset-test-param branch August 31, 2026 04:31
@iamnewton

Copy link
Copy Markdown
Contributor Author

🎉 This PR is included in version 0.5.6 🎉

The release is available on:

Your semantic-release bot 📦🚀

@iamnewton iamnewton added the released Included in a release label Aug 31, 2026
iamnewton added a commit that referenced this pull request Aug 31, 2026
* chore: 🔧 sync from holocron.config

Signed-off-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* chore(release): 0.5.6 [skip ci]

## [0.5.6](v0.5.5...v0.5.6) (2026-08-31)

### Bug Fixes

* **config:** 🐛 merge chromatic and user-flow into react() test param ([#262](#262)) ([9a5c8cf](9a5c8cf)), closes [theholocron/configs#419](theholocron/configs#419)

### Chores

* sync README and repo metadata ([#261](#261)) ([fc99040](fc99040))

Signed-off-by: Newton <5769156+iamnewton@users.noreply.github.com>

* chore: 🔧 fix linting issues

Signed-off-by: super-linterbot[bot] <310376783+super-linterbot[bot]@users.noreply.github.com>

* chore: 🔧 fix linting issues

Signed-off-by: super-linterbot[bot] <310376783+super-linterbot[bot]@users.noreply.github.com>

---------

Signed-off-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Signed-off-by: Newton <5769156+iamnewton@users.noreply.github.com>
Signed-off-by: super-linterbot[bot] <310376783+super-linterbot[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: semantic-release-bot <semantic-release-bot@martynus.net>
Co-authored-by: super-linterbot[bot] <310376783+super-linterbot[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working released Included in a release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant