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

Bump py-evm version requirement #268

Merged
merged 1 commit into from
Jul 26, 2023
Merged

Conversation

kclowes
Copy link
Contributor

@kclowes kclowes commented Jul 20, 2023

What was wrong?

py-evm had some changes that haven't been pulled into eth-tester just yet.

How was it fixed?

Bumped the dependency.

To-Do:

Cute Animal Picture

Cute animal picture

@kclowes kclowes marked this pull request as ready for review July 24, 2023 21:18
@kclowes
Copy link
Contributor Author

kclowes commented Jul 26, 2023

@fselmo I didn't see anything that I should explicitly add tests for between py-evm v0.7.0-a.2 and v0.7.0-a.4, but let me know if there is anything I should test for here!

@kclowes kclowes requested review from fselmo, reedsa and pacrob July 26, 2023 15:52
Copy link
Contributor

@pacrob pacrob left a comment

Choose a reason for hiding this comment

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

lgtm!

@fselmo
Copy link
Contributor

fselmo commented Jul 26, 2023

@fselmo I didn't see anything that I should explicitly add tests for between py-evm v0.7.0-a.2 and v0.7.0-a.4, but let me know if there is anything I should test for here!

Yep I think we're all good there 👍🏼

@kclowes kclowes merged commit c2812b6 into ethereum:master Jul 26, 2023
@kclowes kclowes deleted the bump-py-evm branch July 26, 2023 15:57
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.

3 participants