Skip to content
This repository has been archived by the owner on Feb 26, 2024. It is now read-only.

chore: upgrade ethereumjs packages #3821

Merged
merged 9 commits into from
Nov 10, 2022
Merged

chore: upgrade ethereumjs packages #3821

merged 9 commits into from
Nov 10, 2022

Conversation

MicaiahReid
Copy link
Contributor

@MicaiahReid MicaiahReid commented Oct 19, 2022

No description provided.

@MicaiahReid MicaiahReid marked this pull request as draft October 19, 2022 17:56
@MicaiahReid MicaiahReid changed the title chore: upgrade ethereumjs packages; remove now unneeded ts-ignore chore: upgrade ethereumjs packages Oct 24, 2022
@MicaiahReid MicaiahReid changed the base branch from tech-debt-dudes to combine-ejs-ganache-address November 8, 2022 17:26
@@ -48,18 +48,19 @@
"truffle"
],
"dependencies": {
"@ethereumjs/common": "3.0.0",
"@ethereumjs/common": "3.0.1",
"@ethereumjs/util": "8.0.2",
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have added @ethereumjs/util to this package because I was getting compilation errors. The @ethereumjs/common dependency used stuff from the @ethereumjs/util library, but I believe it was using an older version, which was causing type errors. It was hard to pin down the exact cause, so that reason could be a little off. But this does fix it.

@MicaiahReid MicaiahReid marked this pull request as ready for review November 9, 2022 21:15
Copy link
Contributor

@tenthirtyone tenthirtyone left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 been rolling on this branch for genesis.json work

Base automatically changed from combine-ejs-ganache-address to tech-debt-dudes November 9, 2022 21:41
@MicaiahReid MicaiahReid changed the base branch from tech-debt-dudes to develop November 10, 2022 01:23
@MicaiahReid MicaiahReid changed the base branch from develop to tech-debt-dudes November 10, 2022 01:24
Copy link
Contributor

@jeffsmale90 jeffsmale90 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Like!

@MicaiahReid MicaiahReid merged commit 60af3f2 into tech-debt-dudes Nov 10, 2022
@MicaiahReid MicaiahReid deleted the upgrade-ejs branch November 10, 2022 15:51
MicaiahReid added a commit that referenced this pull request Nov 14, 2022
MicaiahReid added a commit that referenced this pull request Nov 14, 2022
MicaiahReid added a commit that referenced this pull request Nov 14, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants