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

Revert k256 bump #6358

Merged
merged 2 commits into from
Nov 20, 2023
Merged

Revert k256 bump #6358

merged 2 commits into from
Nov 20, 2023

Conversation

kayabaNerve
Copy link
Contributor

This k256 version breaks revm, as documented in
RustCrypto/elliptic-curves#988, which breaks foundry.

While I would instead wait to bump revm, foundry nightly (the only supported version per the provided GH action) is broken and I'd rather correct it ASAP.

Motivation

Foundry nightly is broken.

Solution

Revert the k256 bump until revm supports k256 0.13.2.

This k256 version breaks revm, as documented in
RustCrypto/elliptic-curves#988, which breaks foundry.

While I would instead wait to bump revm, foundry nightly (the only supported
version per the provided GH action) is broken and I'd rather correct it ASAP.
@DaniPopes
Copy link
Member

You'll want to lock it in a manifest otherwise anyone can bump it without even meaning to.

@kayabaNerve
Copy link
Contributor Author

Done

@DaniPopes DaniPopes merged commit a0d1911 into foundry-rs:master Nov 20, 2023
19 of 20 checks passed
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.

2 participants