-
-
Notifications
You must be signed in to change notification settings - Fork 9.3k
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: Update mithril/rax/web-components templates, rework Yarn2 e2e #11354
Conversation
731f2ba
to
bae2cbc
Compare
8d3f2c8
to
c2d6cc7
Compare
0560e86
to
2c4ca1c
Compare
Templates for |
a02248e
to
c03072a
Compare
I updated mithril and web component stories. And everything is 🟢 🎉 @shilman @ndelangen 🎆
|
Using `latest` keyword with yarn 1 resolves to Yarn 1.x but we need Yarn 2.
Example stories for riot need to be updated. Moreover, they should be migrated to riot@4.
…@storybook/cli package
It was previously `{{name}}-v{{version}}` but the `v` isn't needed as we are using `latest` version in many and `vX-lts` for angular.
c03072a
to
a8090fb
Compare
@@ -92,7 +116,12 @@ const initStorybook = async ({ cwd, autoDetect = true, name }: Options) => { | |||
logger.info(`🎨 Initializing Storybook with @storybook/cli`); | |||
try { | |||
const type = autoDetect ? '' : `--type ${name}`; | |||
await exec(`npx -p @storybook/cli sb init --yes ${type}`, { cwd }); | |||
|
|||
const sbCLICommand = useLocalSbCli |
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.
version: '3', | ||
generator: fromDeps('riot@3', 'riot-compiler@3', 'riot-tmpl@3'), | ||
}; | ||
// export const riot: Parameters = { |
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.
We really need to come back to these frameworks :s
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.
LGTM!
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.
🙌
Related to #10797
What I did
E2E:
--use-yarn-2
flag to run tests with Yarn 2--use-local-sb-cli
flag to run E2E tests with local@storybook/cli
packagedocument.createElement(...)
or plain text i.e."<div></div>"
)CLI:
How to test
examples-v2-yarn-2
job