Skip to content

Conversation

@devarsh10
Copy link

@devarsh10 devarsh10 commented Mar 26, 2025

Hello,
Here, I have removed few unused dependencies and devDependencies.

Before

Unused dependencies
* event-stream
* joi
* minimist
* mysql
* node-fetch
Unused devDependencies
* eslint-plugin-jest
* jest-junit
* nyc
* pre-commit
* replace
* sinon
Missing dependencies
* eslint-plugin-import: ./.eslintrc.json
* eslint-plugin-n: ./.eslintrc.json
* eslint-plugin-promise: ./.eslintrc.json

After

 Unused dependencies
* mysql
Unused devDependencies
* jest-junit
* pre-commit
Missing dependencies
* eslint-plugin-import: ./.eslintrc.json
* eslint-plugin-n: ./.eslintrc.json
* eslint-plugin-promise: ./.eslintrc.json

And below is the path I am following to remove a package

  • nvm use -> nvm install -> nvm use

  • npm install

  • Run all the test for the first time only => npm test && npm test:coverage && npm test:integration && npm run dep:check && npm run audit:check && npm run lint

  • npm install

  • npx depcheck

    • npm ls <package-which-you're going to uninstall>
  • npm uninstall <unused-package>

  • npm i

  • Final, pre-commit checks => npm test && npm test:coverage && npm test:integration && npm run dep:check && npm run audit:check && npm run lint

  • All should pass

@devarsh10 devarsh10 force-pushed the chore/remove-dep-pi26 branch from 565d942 to 7d5e2a5 Compare March 28, 2025 09:03
@devarsh10 devarsh10 force-pushed the chore/remove-dep-pi26 branch from bb8777d to 8a5f5de Compare March 31, 2025 12:01
@sonarqubecloud
Copy link

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.

1 participant