Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update to latest chromatic package #2410

Merged
merged 1 commit into from
Nov 7, 2019
Merged

Update to latest chromatic package #2410

merged 1 commit into from
Nov 7, 2019

Conversation

BPScott
Copy link
Member

@BPScott BPScott commented Nov 7, 2019

WHY are these changes introduced?

Keeping up to date, making it easier to do new version bumps in the future now that we use their new package

WHAT is this pull request doing?

Move from storybook-chroma to storybook-chromatic as the package is
being unified

How to 🎩

See that the chroma GH check runs successfully

@github-actions
Copy link
Contributor

github-actions bot commented Nov 7, 2019

💦 Potential splash zone of changes introduced to src/**/*.tsx in this pull request:

No significant changes to src/**/*.tsx were detected.


This comment automatically updates as changes are made to this pull request.
Feedback, troubleshooting: open an issue or reach out on Slack in #polaris-tooling.

@BPScott BPScott requested a review from kaelig November 7, 2019 02:32
@BPScott BPScott added the 🤖Skip Changelog Causes CI to ignore changelog update check. label Nov 7, 2019
Copy link
Contributor

@kaelig kaelig left a comment

Choose a reason for hiding this comment

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

Just a quick question about naming, all looks good otherwise!

resolved "https://registry.yarnpkg.com/cssom/-/cssom-0.4.1.tgz#b24111d236b6dbd00cdfacb5ab67a20473381fe3"
integrity sha512-6Aajq0XmukE7HdXUU6IoSWuH1H6gH9z6qmagsstTiN7cW2FNTsb+J2Chs+ufPgZCsV/yo8oaEudQLrb9dGxSVQ==

cssom@~0.3.6:
Copy link
Contributor

Choose a reason for hiding this comment

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

Two additional versions of cssom, wow!

Copy link
Member Author

Choose a reason for hiding this comment

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

yay transitive dependencies :p

@@ -32,7 +32,7 @@ jobs:
paths:
- node_modules
- run: yarn run test:percy
chromaui:
chroma:
Copy link
Contributor

Choose a reason for hiding this comment

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

Should this job be called "chromatic" for consistency? (I researched the difference between chroma and chromatic, and I am a bit confused)

Copy link
Member Author

Choose a reason for hiding this comment

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

Yeah it's a bit all over the place. Chroma is the company, they make Chromatic (the VRT tool) and the early-access thing ChromaUI (the hosted storybook tool, which just got VRT stuff too), they've decided that two products is silly so once they're done with the early access stuff the "ChromaUI" brand will go away and there will be one product "Chromatic 2.0" which does both storybook hosting and VRT - this explains why this PR changes from storybook-chroma to storybook-chromatic as they've already unified their package so they have one package that handles both products while they're split and eventually storybook-chroma shall go away.

I named this chroma so it matches the name of the check that they add that links to their site, which is "ci/chroma — 264 stories published."

.circleci/config.yml Outdated Show resolved Hide resolved
Move from storybook-chroma to storybook-chromatic as the package is
being unified
@kaelig
Copy link
Contributor

kaelig commented Nov 7, 2019

:shipit:

@BPScott BPScott merged commit 6f3df4e into master Nov 7, 2019
@BPScott BPScott deleted the chroma-update branch November 7, 2019 18:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🤖Skip Changelog Causes CI to ignore changelog update check.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants