@ethereumjs/wallet integration TODO list

[acolytec3]

• initial integration of wallet into monorepo - Monorepo Wallet Library Integration #2729
• Replace all buffer references with Uint8Array Migrate wallet from buffer to Uint8Array  #2739
• Integrate tests into CI Add minimal wallet CI workflow #2738
• Update ethereum-cryptography dep to v2.0.0 Updated ethereum-cryptography to ^2.0.0 #2737
• Clean up docs - Clean up wallet docs and readme #2732
• Update ethers devDependency to v6 - Upgrade ethers to v6 in wallet #2747
• Ensure linter/prettier works correctly with wallet- partly finished by Migrate wallet from buffer to Uint8Array  #2739
  -Review the eslint rules I disabled in Migrate wallet from buffer to Uint8Array  #2739 to see if they can be switched back on
• Switch from mocha to tape in tests Migrate wallet tests to `tape #2742
• Remove conflicting types/node devdependency

[Coollaitar]

Updated dependencies of ethereum-cryptography to v2.0.0 #2734

• Update ethereum-cryptography dep to v2.0.0

[Coollaitar]

Can you please elaborate in which files buffer should be replaced with Uint8Array?

[acolytec3]

Can you please elaborate in which files buffer should be replaced with Uint8Array?

Anywhere buffers are used in this new package, they need to be placed with Uint8Arrays.

[Coollaitar]

Can you please elaborate in which files buffer should be replaced with Uint8Array?

Anywhere buffers are used in this new package, they need to be placed with Uint8Arrays.

Okay, I will make a PR for that.

[Coollaitar]

Updated dependencies of ethereum-cryptography to v2.0.0 #2737

• Update ethereum-cryptography dep to v2.0.0

[gabrocheleau]

Can this be closed @acolytec3 ? Seems like all todos have been completed.

[acolytec3]

We should also clean up the conflicting types/node devDependency. I don't think there's any actual work other than removing the ones from wallet but we should do this so we don't have duplicate conflicting dependencies in our repo.

[gabrocheleau]

We should also clean up the conflicting types/node devDependency. I don't think there's any actual work other than removing the ones from wallet but we should do this so we don't have duplicate conflicting dependencies in our repo.

So just this: #2793 ?

[acolytec3]

We should also clean up the conflicting types/node devDependency. I don't think there's any actual work other than removing the ones from wallet but we should do this so we don't have duplicate conflicting dependencies in our repo.

So just this: #2793 ?

Yessir. I meant to submit this since I had done it locally. Will approve once CI passes.