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
chore: cleanup older dependencies #6482
Conversation
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## unstable #6482 +/- ##
=========================================
Coverage 61.72% 61.72%
=========================================
Files 555 555
Lines 58204 58204
Branches 1839 1839
=========================================
Hits 35925 35925
Misses 22240 22240
Partials 39 39 |
Performance Report✔️ no performance regression detected Full benchmark results
|
packages/db/package.json
Outdated
"it-all": "^3.0.4", | ||
"level": "^8.0.0" | ||
}, | ||
"devDependencies": { | ||
"@lodestar/logger": "^1.16.0" | ||
"@lodestar/logger": "^1.16.0", | ||
"@types/levelup": "^4.3.3" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why do we need to keep @types/levelup
? I don't see this being used
f4c3f29
to
8f14fd8
Compare
8f14fd8
to
7039c8c
Compare
🎉 This PR is included in v1.17.0 🎉 |
Motivation
Remove unused dependencies and move
dev
dependencies to the proper section.Used depcheck to identify all relevant
dependencies
/devDependencies
. We might consider automating those checks down the line.