-
-
Notifications
You must be signed in to change notification settings - Fork 9.4k
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: Generate a new project on init in empty directory #24997
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looking great so far!
972ed44
to
1626045
Compare
82dc7e0
to
6379ff8
Compare
@integrayshaun you asked about how to test? How about adding N sandboxes to the
That way the rest of the test infrastructure can run on these sandboxes. It's a little "heavy" but it's basically making sure that the codepath of instantiating a project directly via an external installers & initializing using the CLI works, rather than our main sandbox mechanism, which is a different codepath and can lag behind external installers by a day or more. WDYT? |
I think the canary release section was deleted in the PR template, that's why it won't get updated. Here's the canary for anyone who wants to try it out:
|
|
Also, appending |
When running:
and choosing angular, when the CLI tries to run Storybook, it fails: The generation works correctly, so if you run Storybook manually later it does work. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Woohooo!!! 🙌
What I did
storybook init
is run on an empty directory, ask user to select a project to generate.Checklist for Contributors
Testing
The changes in this PR are covered in the following automated tests:
Manual testing
yarn task --task compile
.zshrc
Documentation
MIGRATION.MD
Checklist for Maintainers
When this PR is ready for testing, make sure to add
ci:normal
,ci:merged
orci:daily
GH label to it to run a specific set of sandboxes. The particular set of sandboxes can be found incode/lib/cli/src/sandbox-templates.ts
Make sure this PR contains one of the labels below:
Available labels
bug
: Internal changes that fixes incorrect behavior.maintenance
: User-facing maintenance tasks.dependencies
: Upgrading (sometimes downgrading) dependencies.build
: Internal-facing build tooling & test updates. Will not show up in release changelog.cleanup
: Minor cleanup style change. Will not show up in release changelog.documentation
: Documentation only changes. Will not show up in release changelog.feature request
: Introducing a new feature.BREAKING CHANGE
: Changes that break compatibility in some way with current major version.other
: Changes that don't fit in the above categories.🦋 Canary release
This pull request has been released as version
0.0.0-pr-24997-sha-f3766dc1
. Install it by pinning all your Storybook dependencies to that version.More information
0.0.0-pr-24997-sha-f3766dc1
cli/empty-project-init
f3766dc1
1701785787
)To request a new release of this pull request, mention the
@storybookjs/core
team.core team members can create a new canary release here or locally with
gh workflow run --repo storybookjs/storybook canary-release-pr.yml --field pr=24997