-
Notifications
You must be signed in to change notification settings - Fork 101
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 #427
Comments
I can take this issue @arbrandes |
jciasenza
added a commit
to jciasenza/frontend-app-learner-dashboard
that referenced
this issue
Sep 3, 2024
i have made PR #495 |
Thanks @jciasenza! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
There are a handful of package dependencies that appear to be unused by the code in this repository. Some of these just need to be moved to
devDependencies
while others are not direct requirements and can at least be removed from thepackage.json
. For example, thehistory
package is used byfrontend-platform
, but notfrontend-app-learner-dashboard
. So when it's removed from Learner Dashboardpackage.json
, it will only appear in package-lock under frontend-platform, but won't be a direct dependency for Learner Dashboard.The below listed dependencies were chosen based on a simple search through frontend-app-learner-dashboard to see if they were utilized. Any packages that were not imported from or did not appear to be used in some other way are included in the list.
To assess for removal:
To move to
devDependencies
:The text was updated successfully, but these errors were encountered: