Skip to content
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

feat: bump ethers #5086

Merged
merged 6 commits into from
Jun 1, 2023
Merged

feat: bump ethers #5086

merged 6 commits into from
Jun 1, 2023

Conversation

Evalir
Copy link
Member

@Evalir Evalir commented May 31, 2023

Ethers 2.0.6 was recently released, and contains a few useful bugfixes, especially regarding verification.

@Evalir Evalir requested review from gakonst and mattsse May 31, 2023 00:46
@DaniPopes
Copy link
Member

Needs updates on solc breaking changes. Also this makes Shanghai the default EvmVersion, unless we override the default logic somewhere else ourselves.

@Evalir Evalir changed the title chore(packages): bump ethers feat: bump ethers & set Shanghai as default version May 31, 2023
@Evalir Evalir marked this pull request as draft May 31, 2023 17:11
@Evalir Evalir changed the title feat: bump ethers & set Shanghai as default version feat: bump ethers May 31, 2023
@Evalir
Copy link
Member Author

Evalir commented May 31, 2023

@mattsse wdyt of still keeping paris as the default EVM version and overriding ethers's?

@@ -1723,7 +1723,7 @@ impl Default for Config {
allow_paths: vec![],
include_paths: vec![],
force: false,
evm_version: Default::default(),
evm_version: EvmVersion::Paris,
Copy link
Member

Choose a reason for hiding this comment

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

yeah let's keep Paris

@mattsse
Copy link
Member

mattsse commented May 31, 2023

test_issue_2956

there's a no activated error which perhaps is caused by the diff defaults?

@Evalir
Copy link
Member Author

Evalir commented May 31, 2023

Yup, there's also still a few changes left to properly handle leaving paris as default, just testing everything carefully before removing draft

@Evalir Evalir marked this pull request as ready for review May 31, 2023 19:52
@Evalir
Copy link
Member Author

Evalir commented May 31, 2023

alright, so tests fixed @mattsse . I've pinned them to use 0.8.18—if not, we would run into the NotActivated error as forge would try and use 0.8.20 but not have Shanghai and run into invalid opcodes.

Copy link
Member

@mattsse mattsse left a comment

Choose a reason for hiding this comment

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

pinning to 0.8.18 makes sense imo

@mattsse mattsse merged commit 033bdc2 into master Jun 1, 2023
@mattsse mattsse deleted the evalir/bump-ethers branch June 1, 2023 14:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants