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

build(addon-measure): use tsup to build addon-measure and fix related imports in examples/official-storybook #18837

Merged

Conversation

luciana-mendonca
Copy link
Member

Issue: #18732

What I did

  • Migrated addon-measure to use the ts-up bundler
  • Fixed the import paths in examples/official-storybook related to addons/measure, as it was using the compiled output from @storybook/addon-measure/dist/esm/

How to test

  • Is this testable with Jest or Chromatic screenshots? No
  • Does this need a new example in the kitchen sink apps? No
  • Does this need an update to the documentation? No

If your answer is yes to any of these, please make sure to include it in your PR.

Copy link
Member

@ndelangen ndelangen left a comment

Choose a reason for hiding this comment

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

the preview file has exports, they should be re-exported

@ndelangen ndelangen added the maintenance User-facing maintenance tasks label Aug 2, 2022
@ndelangen ndelangen self-assigned this Aug 2, 2022
@ndelangen ndelangen merged commit 4298fbd into storybookjs:next Aug 2, 2022
@ndelangen
Copy link
Member

@luciana-mendonca I invited you to the org! Thank you for your contributions!

@luciana-mendonca
Copy link
Member Author

@ndelangen Thank you very much!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
maintenance User-facing maintenance tasks
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants