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

Reduce number of vulnerabilities and build size. #40

Open
wants to merge 8 commits into
base: update-deps
Choose a base branch
from

Conversation

ExLeonem
Copy link

Moving react and react-dom to peer dependencies reduces the size of the resulting build by about 8 kB.
Updating the outdated npm packages and removing the default docz theme also drastically reduces the number of vulnerabilities.

See Issue #37

cwlsn and others added 8 commits June 4, 2019 20:00
* updated yarn.lock when running 'yarn'

* run an 'npm i'
import yarn.lock from package-lock.json

* update storybook to work with imported yarn.lock
- update storybook configuration
- update docz version
- remove docz default theme, reduce vulnerability issues
- move react and react-dom to peerDependencies
- update webpack config to exclude react and react-dom from build
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants