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

Add AuthCall variant to CallScheme #1367

Closed
wants to merge 1 commit into from

Conversation

tcoratger
Copy link
Contributor

@rakita
Copy link
Member

rakita commented May 1, 2024

@Rjected does this enum need to be used in the handler in Alphanet to be useful?

@Rjected
Copy link
Collaborator

Rjected commented May 3, 2024

@Rjected does this enum need to be used in the handler in Alphanet to be useful?

It's not used in the handler, no, I'm not sure if this is needed actually

@Rjected
Copy link
Collaborator

Rjected commented May 3, 2024

it looks like it exists in the branch of revm used in alphanet-foundry:
https://github.com/wevm/revm/blob/66a499fc281b493a34ee42069309cf7aed363052/crates/interpreter/src/inner_models.rs#L109

warning: old revm

Still need to understand how this is used to determine if it's necessary

@rakita
Copy link
Member

rakita commented May 14, 2024

I will close this for now, and we can always revisit it in future.

There is possibility that EIP-3074 is not going in

@rakita rakita closed this May 14, 2024
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

4 participants