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 dependencies #7

Closed
aniketnk opened this issue Jul 20, 2020 · 3 comments · Fixed by #10
Closed

Remove unused dependencies #7

aniketnk opened this issue Jul 20, 2020 · 3 comments · Fixed by #10
Labels
enhancement New feature or request

Comments

@aniketnk
Copy link
Collaborator

Is your feature request related to a problem? Please describe.
There are many ununsed dependencies from package.json. Add them only as necessary.
Used the depcheck library to find them.

$ depcheck
Unused dependencies
* errorhandler
* nodemailer
* nodemailer-express-handlebars
Unused devDependencies
* @types/errorhandler
* @types/node
* @types/shelljs
* eslint-config-prettier
* eslint-config-strongloop
* eslint-plugin-prettier
* shelljs
* ts-node

Describe the solution you'd like
Use depcheck and remove unnecessary packages.

Describe alternatives you've considered
None

Additional context
Add any other context or screenshots about the feature request here.

@aniketnk
Copy link
Collaborator Author

Should apply to boilerplate once #1 is merged.

@aniketnk
Copy link
Collaborator Author

@temanisparsh What's the status on this?

@aniketnk aniketnk added the enhancement New feature or request label Jul 22, 2020
@temanisparsh
Copy link
Contributor

Will push changes in a while

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

Successfully merging a pull request may close this issue.

2 participants