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

Make "Accepted" definitions consistent for Core EIPs #1858

Closed
wants to merge 2 commits into from
Closed

Make "Accepted" definitions consistent for Core EIPs #1858

wants to merge 2 commits into from

Conversation

timbeiko
Copy link
Contributor

@timbeiko timbeiko commented Mar 22, 2019

EIP-1 has two conflicting definitions of what "Accepted" meant for Core EIPs. This harmonizes them.

The previous version of EIP-1 had two conflicting definitions of what "Accepted" meant for Core EIPs. This harmonizes them.
@timbeiko timbeiko changed the title Make "Accepted" definition consistent for Core EIPs Make "Accepted" definitions consistent for Core EIPs Mar 22, 2019
Copy link
Contributor

@Arachnid Arachnid left a comment

Choose a reason for hiding this comment

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

Core EIPs can be final.

EIPS/eip-1.md Show resolved Hide resolved
EIPS/eip-1.md Outdated Show resolved Hide resolved
Copy link
Contributor

@Arachnid Arachnid left a comment

Choose a reason for hiding this comment

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

LGTM

@timbeiko
Copy link
Contributor Author

@Arachnid great, thanks! What is the process for merging it in? Do you have write access?

Copy link
Contributor

@nicksavers nicksavers left a comment

Choose a reason for hiding this comment

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

I might be missing something, but I don't think this should be merged. The intermediate points don't explain what the status means, but what is needed to get to that status.

@timbeiko timbeiko closed this Apr 1, 2019
@timbeiko
Copy link
Contributor Author

timbeiko commented Apr 1, 2019

@nicksavers thanks for the feedback. I think your point makes sense and I’ll close the first PR.

@timbeiko timbeiko deleted the patch-1 branch April 1, 2019 20:25
Arachnid pushed a commit that referenced this pull request Apr 1, 2019
* Updated README to be consistent with EIP-1

Update README to be consistent with EIP and this PR: #1858

* Updated index.html 

Updated index.html to use the same definitions as the readme.
ilanolkies pushed a commit to ilanolkies/EIPs that referenced this pull request Nov 12, 2019
)

* Updated README to be consistent with EIP-1

Update README to be consistent with EIP and this PR: ethereum#1858

* Updated index.html 

Updated index.html to use the same definitions as the readme.
enjin-io pushed a commit to enjin-io/EIPs-1 that referenced this pull request Nov 13, 2023
* Updated README to be consistent with EIP-1

Update README to be consistent with EIP and this PR: ethereum/EIPs#1858

* Updated index.html 

Updated index.html to use the same definitions as the readme.
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.

3 participants