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

Move EIP-2098 to final call #4611

Merged
merged 9 commits into from
Feb 4, 2022
Merged

Move EIP-2098 to final call #4611

merged 9 commits into from
Feb 4, 2022

Conversation

ricmoo
Copy link
Contributor

@ricmoo ricmoo commented Dec 27, 2021

Moving EIP-2098 to final call.

ricmoo added 6 commits June 22, 2021 16:07

Verified

This commit was signed with the committer’s verified signature. The key has expired.
ricmoo Richard Moore
… review.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
@eth-bot
Copy link
Collaborator

eth-bot commented Dec 27, 2021

All tests passed; auto-merging...

(pass) eip-2098.md

classification
updateEIP
  • passed!

Copy link
Member

@lightclient lightclient left a comment

Choose a reason for hiding this comment

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

  • s/Test Vectors/Test Cases
  • Gas Analysis is not a valid ## section per EIP-1, please include that information in one of the other sections. Rationale seems like the best choice?
  • s/Implementations/Reference Implementation
  • Acknowledgments is not a valid EIP section, please remove
  • Security Considerations is missing

@ricmoo
Copy link
Contributor Author

ricmoo commented Jan 22, 2022

These changes have been made.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
Co-authored-by: Micah Zoltu <micah@zoltu.net>

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
@alita-moore alita-moore closed this Feb 4, 2022
@alita-moore alita-moore reopened this Feb 4, 2022
@eth-bot eth-bot enabled auto-merge (squash) February 4, 2022 21:10
@eth-bot eth-bot merged commit 6b5fb88 into ethereum:master Feb 4, 2022
PowerStream3604 pushed a commit to PowerStream3604/EIPs that referenced this pull request May 19, 2022

Unverified

This commit is not signed, but one or more authors requires that any commit attributed to them is signed.
* Updated EIP-2098 compact representation with suggestions and moved to review.

* Move EIP-2098 to final call.

* Updated EIP-2098 as per request.

* Update EIPS/eip-2098.md

Co-authored-by: Micah Zoltu <micah@zoltu.net>

* Update EIPS/eip-2098.md

Co-authored-by: Micah Zoltu <micah@zoltu.net>
Co-authored-by: lightclient <14004106+lightclient@users.noreply.github.com>
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

7 participants