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

Update EIP-2930: Update eip-2930.md #8304

Closed
wants to merge 1 commit into from

Conversation

sambacha
Copy link
Contributor

Add information about the RPC Method and clarify what you SHOULD NOT include in access lists!

@sambacha sambacha requested a review from eth-bot as a code owner March 11, 2024 20:16
@github-actions github-actions bot added c-update Modifies an existing proposal s-final This EIP is Final t-core labels Mar 11, 2024
@eth-bot
Copy link
Collaborator

eth-bot commented Mar 11, 2024

File EIPS/eip-2930.md

Requires 1 more reviewers from @holiman, @vbuterin
Requires 3 more reviewers from @axic, @g11tech, @gcolvin, @lightclient, @SamWilsn, @xinbenlv

@eth-bot eth-bot added the e-consensus Waiting on editor consensus label Mar 11, 2024
@eth-bot eth-bot changed the title Update eip-2930.md Update EIP-2930: Update eip-2930.md Mar 11, 2024
Copy link

The commit b28d27c (as a parent of e8a1b64) contains errors.
Please inspect the Run Summary for details.

@github-actions github-actions bot added the w-ci Waiting on CI to pass label Mar 11, 2024
@lightclient
Copy link
Member

Sorry this EIP is already final. If you want to add info about rpc methods, please direct those PRs to https://github.com/ethereum/eth1.0-[apis](https://github.com/ethereum/execution-apis)

@sambacha
Copy link
Contributor Author

Sorry this EIP is already final. If you want to add info about rpc methods, please direct those PRs to https://github.com/ethereum/eth1.0-apis

This is a non-normative clarification, this type of edit is allowed under EIP-1. I can see how the additional information about the RPC method may be irrelevant, though that should be referenced in an Errata section. 

can @SamWilsn @xinbenlv elaborate on the updating guidelines?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
c-update Modifies an existing proposal e-consensus Waiting on editor consensus s-final This EIP is Final t-core w-ci Waiting on CI to pass
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants