web3.js icon indicating copy to clipboard operation
web3.js copied to clipboard

1.x dependencies update

Open jdevcs opened this issue 3 years ago • 1 comments

  • [ ] uuid
  • [ ] ethers
  • [ ] ethereumjs/libs
  • [ ] and discuss, list and update other libs that can be updated without any breaking changes

jdevcs avatar Sep 08 '22 14:09 jdevcs

check if following required:

  • [ ] Bump terser from 4.8.0 to 4.8.1

jdevcs avatar Sep 20 '22 15:09 jdevcs

So one problem we have with upgrading dependencies is @ethereumjs/common and by extension, @ethereumjs/tx (since it depends on the former)

The problem: In this PR (specifically this commit) the typescript version is bumped from 3.x.x to 4.x.x. After this change, Web3.js starts to get the following tsc error when building web3-eth-accounts:

lerna ERR! npm run compile exited 1 in 'web3-eth-accounts'
lerna ERR! npm run compile stdout:

> [email protected] compile
> tsc -b tsconfig.json

Error: ../../node_modules/@ethereumjs/common/dist/types.d.ts(38,32): error TS1005: ',' expected.
Error: ../../node_modules/@ethereumjs/common/dist/types.d.ts(38,58): error TS1005: ',' expected.
Error: ../../node_modules/@ethereumjs/common/dist/types.d.ts(40,12): error TS1005: ',' expected.
Error: ../../node_modules/@ethereumjs/common/dist/types.d.ts(41,9): error TS1005: ',' expected.
Error: ../../node_modules/@ethereumjs/common/dist/types.d.ts(42,12): error TS1005: ',' expected.
lerna ERR! npm run compile exited 1 in 'web3-eth-accounts'
lerna WARN complete Waiting for 2 child processes to exit. CTRL-C to exit immediately.
Error: Process completed with exit code 1.

Currently in 1.x, the version 2.5.0 of @ethereumjs/common is being used (and version 3.3.2 of @ethereumjs/tx) which was released a year ago according to NPM:

image

PR #5529 updates some dependencies while locking @ethereumjs/common and @ethereumjs/tx to versions 2.5.0 and 3.3.2 respectively, and these errors are no longer received and all tests still pass

PR #5530 accepted to update typescript to ^4.8.4, but received many errors

spacesailor24 avatar Oct 13 '22 03:10 spacesailor24