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

Align with Storybook 8.2 core package layout #330

Merged
merged 12 commits into from
Sep 9, 2024
Merged

Align with Storybook 8.2 core package layout #330

merged 12 commits into from
Sep 9, 2024

Conversation

ndelangen
Copy link
Member

@ndelangen ndelangen commented Jul 8, 2024

This updates the addon to follow "core package consolidation" changes. Various internal Storybook packages have been moved, so this updates the imports.

📦 Published PR as canary version: 3.0.0--canary.330.a7fb32f.0

✨ Test out this PR locally via:

npm install @chromatic-com/[email protected]
# or 
yarn add @chromatic-com/[email protected]

@ndelangen ndelangen self-assigned this Jul 8, 2024
@ndelangen ndelangen requested a review from ghengeveld July 8, 2024 13:59
@ndelangen ndelangen added DO NOT MERGE Block: Prevent merging this PR dependencies Pull requests that update a dependency file labels Jul 8, 2024
@ndelangen ndelangen changed the title update to follow cpc guidelines CPC: update to follow cpc guidelines Jul 8, 2024
@ndelangen ndelangen changed the title CPC: update to follow cpc guidelines CPC: Update to follow cpc guidelines Jul 8, 2024
@ndelangen ndelangen added the minor Auto: Increment the minor version when merged label Jul 8, 2024
@ndelangen
Copy link
Member Author

The visual changes picked up seem to be flake. somehow?

When running on my local machine, the tooltips show up.

@ghengeveld
Copy link
Member

Yeah those hover tooltips are flaky. I've denied and rerun the build. If the new build is the same then I will accept them.

@ndelangen
Copy link
Member Author

ndelangen commented Jul 8, 2024

Do not merge until 8.2 is fully released.

This change would lock us into supporting only 8.2+ (unsure if that's what we want to do?)

I opened this mainly to showcase what needs to happen to be CPC "compliant"

Copy link
Member

@ghengeveld ghengeveld left a comment

Choose a reason for hiding this comment

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

Needs to be updated to the actual ^8.2.0 range but other than that this is good.

@ndelangen ndelangen removed the DO NOT MERGE Block: Prevent merging this PR label Aug 29, 2024
@ghengeveld ghengeveld added the skip-release Auto: Preserve the current version when merged label Sep 6, 2024
@ghengeveld ghengeveld changed the title CPC: Update to follow cpc guidelines Align with new Storybook package layout Sep 6, 2024
@ghengeveld ghengeveld changed the title Align with new Storybook package layout Align with new Storybook core package layout Sep 6, 2024
@ghengeveld ghengeveld changed the title Align with new Storybook core package layout Align with Storybook 8.2 core package layout Sep 6, 2024
@ghengeveld ghengeveld added major Auto: Increment the major version when merged and removed minor Auto: Increment the minor version when merged labels Sep 9, 2024
@ghengeveld ghengeveld added release Auto: Create a `latest` release when merged and removed skip-release Auto: Preserve the current version when merged labels Sep 9, 2024
@ghengeveld ghengeveld merged commit 63bd4f7 into main Sep 9, 2024
11 of 12 checks passed
@ghengeveld ghengeveld deleted the norbert/cpc branch September 9, 2024 12:38
@yannbf yannbf restored the norbert/cpc branch September 11, 2024 21:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file major Auto: Increment the major version when merged release Auto: Create a `latest` release when merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants