-
-
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
Switch to monorepo #749
Switch to monorepo #749
Conversation
Fix wrong npm package path
Provide storybook context to render
Add support for matching a story against a regex to run a story
Sorted stories by kind
Updated README to include sortStoriesByKind option
Provide mock Storybook Channel to Storybook Addons for compatibility with Storybook Addon Knobs
addons might init expecting the channel to exist.
@ndelangen Here are some notes from running it on a local project. TLDR; worked for me after a few minor bumps. Looks good!
|
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.
Looks good! See my comments above, and also I agree with @tmeasday about merging early.
This message is a known issue: https://github.com/storybooks/mono/issues/19 |
.eslintrc
Outdated
"printWidth": 100, | ||
"tabWidth": 2, | ||
"bracketSpacing": true, | ||
# "trailingComma": "es5", |
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.
Do we want to decide on this comma rule? I vote for all
as a value
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.
done
.eslintrc
Outdated
}], | ||
"quotes": ["warn", "single"], | ||
"arrow-parens": ["warn", "as-needed"] | ||
# "comma-dangle": ["warn", "always"] |
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.
Do we want to remove this line?
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.
done
# Conflicts: # packages/react-storybook/src/server/config/defaults/webpack.config.js # packages/react-storybook/src/server/middleware.js
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.
+1 for mergin
☁️ Nx Cloud ReportCI is running/has finished running commands for commit 2df4be7. As they complete they will appear below. Click to see the status, the terminal output, and the build insights. 📂 See all runs for this CI Pipeline Execution ✅ Successfully ran 1 targetSent with 💌 from NxCloud. |
Issue: #742
What I did
Moved all the source for this package into a /packages/react-storybook
Merged in the monorepo master
Resolved mergeconflicts
How to test
install dependencies - this should bootstrap lerna.
run
npm run test
in the root