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

Remove unused npm dependencies #620

Closed
wants to merge 54 commits into from
Closed

Conversation

ChiaMineJP
Copy link
Contributor

@ChiaMineJP ChiaMineJP commented Jan 14, 2022

See #607 for detail.

Memo

@chia/gui

(1) Remove @material-styles byte-size npm-audit-resolver stream-browserify from packages/gui/package.json by hand.
(2) npm run clean -- --scope @chia/gui
(3) npm run bootstrap

# of dependencies in package-lock.json of @chia/gui before this PR: 1060
After this PR: 1043

@chia/wallet

(1) Remove @material-styles byte-size npm-audit-resolver polished stream-browserify stringify tail yarn from packages/wallet/package.json by hand.
(2) npm run clean -- --scope @chia/wallet
(3) npm run bootstrap

# of dependencies in package-lock.json of @chia/wallet before this PR: 1042
After this PR: 943

@chia/core

(1) Remove cookie core-js isomorphic-fetch qs @types/core-js @types/isomorphic-fetch @types/qs from packages/core/package.json by hand.
(2) npm run clean -- --scope @chia/core
(3) npm run bootstrap

# of dependencies in package-lock.json of @chia/wallet before this PR: 368
After this PR: 354

Additional Note

  • @material-ui/lab is not used in @chia/wallet but removing it from @chia/wallet's package.json results in an error because @chia/core specifies @material-ui/lab as peerDependencies.

@ChiaMineJP ChiaMineJP mentioned this pull request Jan 14, 2022
1 task
@github-actions
Copy link
Contributor

'This PR has been flagged as stale due to no activity for over 60
days. It will not be automatically closed, but it has been given
a stale-pr label and should be manually reviewed.'

@github-actions
Copy link
Contributor

'This PR has been flagged as stale due to no activity for over 60
days. It will not be automatically closed, but it has been given
a stale-pr label and should be manually reviewed.'

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants