-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
fix: ethereumjs-util -> @ethereumjs/util@^8.1.0 #11979
base: main
Are you sure you want to change the base?
Conversation
CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes. |
New and removed dependencies detected. Learn more about Socket for GitHub ↗︎
🚮 Removed packages: npm/@types/bn.js@5.1.5, npm/bindings@1.5.0, npm/bip66@1.1.5, npm/drbg.js@1.0.1, npm/file-uri-to-path@1.0.0 |
3c97a52
to
88144dc
Compare
Bitrise✅✅✅ Commit hash: 88144dc Note
|
88144dc
to
a6e2bb1
Compare
Bitrise✅✅✅ Commit hash: a6e2bb1 Note
|
a6e2bb1
to
bbedc78
Compare
bbedc78
to
6562082
Compare
- to get correct types - to prepare upgrading to versions which don't provide the re-export - fix: vendor BNToHex and hexToBN in order to use bn.js v4 - fix: explicitly use either bn.js@4 and bn.js@5 - Why mixing v4 and v5 is A Bad Thing: ethereumjs/ethereumjs-util#250
6562082
to
391bceb
Compare
Bitrise❌❌❌ Commit hash: 391bceb Note
Tip
|
Quality Gate passedIssues Measures |
Description
Replaces legacy
ethereumjs-util
with maintained@ethereumjs/util
.Related issues
Based on
Manual testing steps
Screenshots/Recordings
Before
After
Pre-merge author checklist
Pre-merge reviewer checklist