-
Notifications
You must be signed in to change notification settings - Fork 5.3k
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
Abandon EIP-1706 #2543
Abandon EIP-1706 #2543
Conversation
ca7d72c
to
06613c3
Compare
Hi! I'm a bot, and I wanted to automerge your PR, but couldn't because of the following issue(s):
|
EIPS/eip-1706.md
Outdated
@@ -3,12 +3,14 @@ eip: 1706 | |||
title: Disable SSTORE with gasleft lower than call stipend | |||
author: Alex Forshtat <alex@tabookey.com>, Yoav Weiss <yoav@tabookey.com> | |||
discussions-to: https://github.com/alex-forshtat-tbk/EIPs/issues/1 | |||
status: Draft | |||
status: Superseded |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
According to EIP-1 "Superseded" applies to "Final" EIPs, e.g. the fitting one here would be "Abandoned" with perhaps a note that EIP-2200 is the preferred way.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changed status to 'abandoned'
* Mark 1706 as 'superseded' instead of 'draft' * Change 'superseded' to 'abandoned'
* Mark 1706 as 'superseded' instead of 'draft' * Change 'superseded' to 'abandoned'
* Mark 1706 as 'superseded' instead of 'draft' * Change 'superseded' to 'abandoned'
While 1706 was never 'final' on its own, I think keeping it as 'draft', or marking it 'abandoned' or 'rejected' creates a false perception that the logic specified in it never made it to the EVM.
EIP-2200 explicitly states it includes all provisions of this EIP word-by-word.