-
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
Automatically merge all new EIP drafts #5641
Comments
This seems reasonable. I'd like to have a bit of discussion on it before we make a decision though. I think we need a few more checks in
We'd also need an automated EIP number assigner. |
I agree that those specific checks should be required. |
@poojaranjan brought up a great point. A lot of EIPs die in the draft stage, so the first pull request is the only point editors have to influence these EIPs. Plus we want to encourage discussion on magicians before draft, so we get bigger/more developed ideas in the repository. I am now weakly against this proposal. |
I agree. |
I have to disagree here. Editors can always open PRs to modify EIPs (like any other person). I think that having draft EIPs automatically merge would encourage more participation for draft EIPs, as it gives the Ethereum community a longer time to suggest and make changes formally. As such, the quality of Draft EIPs, on the whole, would be lower, but the quality of EIPs moving to Review would be higher as people would have more time to participate in the horizontal review process. |
Would like to bump this. |
IMO, if we're going to "auto-merge" drafts they should not get an EIP number. Other standards bodies have a draft stage where the proposal has not yet been assigned an RFC number (see Internet-Drafts). I would like to adopt a similar scheme. I think this would be valuable regardless if we move forward with the auto-merge. |
I would agree with them not getting an EIP number. And if they become stagnant, I think they could just be deleted. +1 |
+1 but in that case we need to suggest a standard way for them to create file name with, for example |
Nothing says that EIP "numbers" even have to be numeric. EIP editors, right now, could literally assign an EIP "number" of CC @SamWilsn, how feasible would this be to add to EIPw? |
I think they should have a different prefix completely to avoid any confusion. |
How about |
Sounds good.
|
I was thinking of a mnemonic, like I-Ds. |
Mnmonecs sounds good to me. Better than a draft number, reducing chance of confusion with regular EIP number |
Yes good, kill them while the gestate. If you do not have the time to shepherd it through a GitHub pull request what chance do they have of championing it through all the factions in the greater community? Plus why make more work for yourself, lol. |
That's the problem I have with having two bars: you are doubling the workload. You have to review it once during draft, and again when it reaches review. The difference is that Review EIPs are typically nicer to review, having been somewhat proofread and shortened. So why not get rid of that first step? |
There has been no activity on this issue for 1 week. It will be closed after 3 months of inactivity. |
Dismissing stale bot. |
There has been no activity on this issue for 1 week. It will be closed after 3 months of inactivity. |
Dismissing stale bot. |
It will not. It will assign them a string identifier. |
There has been no activity on this issue for 1 week. It will be closed after 3 months of inactivity. |
Dismissing stale bot |
There has been no activity on this issue for 1 week. It will be closed after 3 months of inactivity. |
Dismissing stale bot |
There has been no activity on this issue for 1 week. It will be closed after 3 months of inactivity. |
This is still being discussed is it not? |
Yes. The bot is to prompt us to either continue discussion or let it stagnate. I would, again, very much like this. |
Again, I think they should have a different prefix completely to avoid any confusion and gaming of the system. |
I am +0 to a different prefix (not +1 because I would prefer a mnemonic instead, which would require far fewer CI changes) |
There has been no activity on this issue for 1 week. It will be closed after 3 months of inactivity. |
Bump |
There has been no activity on this issue for 1 week. It will be closed after 3 months of inactivity. |
Draft EIP numbers are being made possible by #6976 |
There has been no activity on this issue for 1 week. It will be closed after 3 months of inactivity. |
This issue was closed due to inactivity. If you are still pursuing it, feel free to reopen it and respond to any feedback. |
Proposed Change
Currently, the process for creating an EIP is not very simple and it is pretty time-consuming just to get an EIP to draft. I imagine a large part of the reason that a lot of EIPs end up stagnant is that the effort even to get an EIP into draft is a lot - more than is probably needed when the criteria for drafts, per EIP-1, is them being "properly formatted." We have tooling that lets users know when their EIPs aren't properly formatted, and we might soon have tooling to automatically fix the most common errors.
Currently, when an EIP is initially proposed is when things like "is this EIP ideal for its current purpose" are discussed and block the progress. Having an actual EIP that is being built and on which people can submit PRs that propose changes is, IMHO, better than the status quo, and would actually allow the community to take some of the burden off of the EIP editors.
The text was updated successfully, but these errors were encountered: