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

CLI: overhaul storybook repro command and rename it to storybook sandbox #20507

Merged
merged 5 commits into from
Jan 16, 2023

Conversation

yannbf
Copy link
Member

@yannbf yannbf commented Jan 5, 2023

Issue: N/A

What I did

The sb repro as we know it was entirely removed, as it's been "deprecated" for some time.
The former sb repro-next command becomes sb sandbox (and is aliased as repro for compatibility purposes), with the updated functionality bringing the correct sandboxes.

TODO:

  • rename repro-templates-temp repository to sandboxes
  • rename everything from repro to sandbox

How to test

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

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

@yannbf yannbf self-assigned this Jan 5, 2023
@yannbf yannbf force-pushed the chore/make-repro-next-default-command branch from 567e3da to 02f9492 Compare January 10, 2023 09:29
@yannbf yannbf added the ci:merged Run the CI jobs that normally run when merged. label Jan 10, 2023
@yannbf yannbf marked this pull request as ready for review January 10, 2023 09:29
@ndelangen ndelangen self-requested a review January 10, 2023 11:15
@yannbf yannbf force-pushed the chore/make-repro-next-default-command branch from 02f9492 to 2737195 Compare January 10, 2023 11:26
@yannbf yannbf force-pushed the chore/make-repro-next-default-command branch from 0894072 to f7b089b Compare January 12, 2023 11:27
@yannbf
Copy link
Member Author

yannbf commented Jan 12, 2023

The failures of this PR are related to a recent regression from next, which got introduced here after I rebased. Gotta check that

- sb repro as it was is entirely removed
- sb repro-next becomes sb repro
- replace screenshots with new sb repro command screenshots
@yannbf yannbf force-pushed the chore/make-repro-next-default-command branch from f7b089b to 46bbd21 Compare January 12, 2023 14:47
Copy link
Member

@tmeasday tmeasday left a comment

Choose a reason for hiding this comment

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

Works great, looks great!

@yannbf yannbf changed the title CLI: overhaul repro command CLI: overhaul storybook repro command and rename it to storybook sandbox Jan 16, 2023
@yannbf yannbf merged commit de2755e into next Jan 16, 2023
@yannbf yannbf deleted the chore/make-repro-next-default-command branch January 16, 2023 10:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci:merged Run the CI jobs that normally run when merged. cli feature request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants