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

fix: bump revm-inspectors to 0.7.4 #8896

Closed
wants to merge 1 commit into from

Conversation

iajoiner
Copy link

@iajoiner iajoiner commented Sep 18, 2024

Motivation

Fixes #8895. Upstream semver issue in alloy caused downstream issues in proof-of-sql which uses forge-script which requires us to bump revm-inspectors to 0.7.4.

Solution

See above.

@iajoiner iajoiner changed the title feat: bump revm-inspectors to 0.7.4 fix: bump revm-inspectors to 0.7.4 Sep 18, 2024
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.

fyi @yash-atreya re alloy bump

@mattsse
Copy link
Member

mattsse commented Sep 18, 2024

messed up semver due to reasons...
sorry about this will be taken care of asap

@iajoiner
Copy link
Author

@iajoiner iajoiner force-pushed the bump-revm-inspectors branch 3 times, most recently from cb03613 to 0c2ad2b Compare September 19, 2024 08:33
@iajoiner
Copy link
Author

Please approve the workflow again. Thanks!

@iajoiner
Copy link
Author

Closed in favor of #8902 .

@iajoiner iajoiner closed this Sep 19, 2024
@iajoiner iajoiner deleted the bump-revm-inspectors branch September 19, 2024 08:36
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.

bug: Need to bump revm-inspectors to 0.7.4
2 participants