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

Refactor : Identify and Remove Unused Dependencies #204

Open
Abhijay007 opened this issue Apr 12, 2024 · 3 comments
Open

Refactor : Identify and Remove Unused Dependencies #204

Abhijay007 opened this issue Apr 12, 2024 · 3 comments

Comments

@Abhijay007
Copy link
Contributor

Issue Description:

As we are currently in the process of upgrading project dependencies, it's essential to assess our dependency usage thoroughly. There could be instances where certain dependencies are included but remain unused. For instance, although Redux is present in our codebase, it's not actively utilized. Identifying and removing such dependencies can help streamline the project, improve maintainability, and reduce potential security risks.

Action Items:

  • Conduct a comprehensive review of the project dependencies to identify unused ones.
  • Create a systematic approach or tooling to assist in identifying unused dependencies.
  • Document the process of identifying and removing unused dependencies for future reference.
  • Test the project thoroughly after removing unused dependencies to ensure no functionality is impacted.

Additional Notes:

This issue serves as a tracking mechanism for the task of identifying and removing unused dependencies. Any related discussions, findings, or updates should be documented here. Let's ensure our project remains lean, efficient, and up-to-date with only the necessary dependencies.

Note for contributors: This is an open issue, and needs no assignment. Contributors can just comment below on which dependency they are working on and create a PR for the same.

@AdiAkhileshSingh15
Copy link
Contributor

Hi @Abhijay007 👋🏻 ! I'm interested in helping resolve this issue. However, I'd like to ensure we have @lrasmus' approval before removing any unnecessary or unused dependencies, taking into account their potential usefulness in the future.
I'll comment a list of the dependencies I propose to remove with my PR once we have the green light. 😄

@Abhijay007
Copy link
Contributor Author

Hi @Abhijay007 👋🏻 ! I'm interested in helping resolve this issue. However, I'd like to ensure we have @lrasmus' approval before removing any unnecessary or unused dependencies, taking into account their potential usefulness in the future. I'll comment a list of the dependencies I propose to remove with my PR once we have the green light. 😄

Hi @AdiAkhileshSingh15 you can go ahead with this issue, I already discussed this issue with @lrasmus, and ya it's great if you could also list the unnecessary or unused dependencies you are removing

@AdiAkhileshSingh15
Copy link
Contributor

I'm working on removing the following dependencies:

devDependencies:

  1. enzyme-to-json
  2. fbjs-scripts
  3. redux-logger
  4. testcafe-browser-provider-electron
  5. testcafe-react-selectors

dependencies:

  1. ansi-regex
  2. axios
  3. connected-react-router
  4. dt-python-parser
  5. object-hash
  6. react-redux
  7. react-select
  8. redux
  9. redux-thunk

Will soon add a PR to address these. 🙌🏻

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

No branches or pull requests

2 participants