EIPs icon indicating copy to clipboard operation
EIPs copied to clipboard

Automatically merge all new EIP drafts

Open Pandapip1 opened this issue 2 years ago • 5 comments

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.

Pandapip1 avatar Sep 10 '22 17:09 Pandapip1

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 eipw before I'm comfortable enabling this:

  • ethereum/eipw#11
  • ethereum/eipw#27

We'd also need an automated EIP number assigner.

SamWilsn avatar Sep 12 '22 04:09 SamWilsn

I agree that those specific checks should be required.

Pandapip1 avatar Sep 12 '22 12:09 Pandapip1

@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.

SamWilsn avatar Sep 21 '22 14:09 SamWilsn

I agree.

xinbenlv avatar Oct 27 '22 13:10 xinbenlv

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.

Pandapip1 avatar Oct 27 '22 13:10 Pandapip1

Would like to bump this.

Pandapip1 avatar Nov 25 '22 16:11 Pandapip1

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.

lightclient avatar Nov 28 '22 13:11 lightclient

I would agree with them not getting an EIP number. And if they become stagnant, I think they could just be deleted. +1

Pandapip1 avatar Nov 28 '22 17:11 Pandapip1

+1

but in that case we need to suggest a standard way for them to create file name with, for example some mnemonic name, e.g. eip-some-name.md

xinbenlv avatar Nov 28 '22 17:11 xinbenlv

Nothing says that EIP "numbers" even have to be numeric. EIP editors, right now, could literally assign an EIP "number" of hello. We wouldn't since that would be stupid, but we could do this without too many changes.

CC @SamWilsn, how feasible would this be to add to EIPw?

Pandapip1 avatar Nov 28 '22 17:11 Pandapip1

I think they should have a different prefix completely to avoid any confusion.

lightclient avatar Nov 29 '22 16:11 lightclient

I think they should have a different prefix completely to avoid any confusion.

How about EIP-draft-X and ERC-draft-X?

Pandapip1 avatar Dec 02 '22 20:12 Pandapip1

How about EIP-draft-X and ERC-draft-X?

Sounds good.

X could be ether a number or a mnemonic alias string

xinbenlv avatar Dec 02 '22 20:12 xinbenlv

I was thinking of a mnemonic, like I-Ds.

Pandapip1 avatar Dec 02 '22 20:12 Pandapip1

Mnmonecs sounds good to me. Better than a draft number, reducing chance of confusion with regular EIP number

xinbenlv avatar Dec 02 '22 20:12 xinbenlv

@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.

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.

sambacha avatar Dec 19 '22 07:12 sambacha

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?

Pandapip1 avatar Dec 20 '22 18:12 Pandapip1

There has been no activity on this issue for 1 week. It will be closed after 3 months of inactivity.

github-actions[bot] avatar Jan 26 '23 00:01 github-actions[bot]

Dismissing stale bot.

Pandapip1 avatar Jan 26 '23 18:01 Pandapip1

There has been no activity on this issue for 1 week. It will be closed after 3 months of inactivity.

github-actions[bot] avatar Feb 03 '23 00:02 github-actions[bot]

Dismissing stale bot.

Pandapip1 avatar Feb 04 '23 00:02 Pandapip1

Honestly it would be great for drafts to be in a completely separate repo, decrease the traffic to the main repo.

lightclient avatar Feb 08 '23 14:02 lightclient

Honestly it would be great for drafts to be in a completely separate repo, decrease the traffic to the main repo.

How about a completely separate folder?

Pandapip1 avatar Feb 08 '23 14:02 Pandapip1

There has been no activity on this issue for 1 week. It will be closed after 3 months of inactivity.

github-actions[bot] avatar Feb 16 '23 00:02 github-actions[bot]

Waiting on #6496

Pandapip1 avatar Feb 16 '23 14:02 Pandapip1

I am very against this change if it assigns them an EIP number. I think it should give a DEIP number or something separate.

lightclient avatar Feb 16 '23 14:02 lightclient

There has been no activity on this issue for 1 week. It will be closed after 3 months of inactivity.

github-actions[bot] avatar Feb 25 '23 00:02 github-actions[bot]

I am very against this change if it assigns them an EIP number. I think it should give a DEIP number or something separate.

It will not. It will assign them a string identifier.

Pandapip1 avatar Feb 25 '23 01:02 Pandapip1

There has been no activity on this issue for 1 week. It will be closed after 3 months of inactivity.

github-actions[bot] avatar Mar 05 '23 00:03 github-actions[bot]

Dismissing stale bot

Pandapip1 avatar Mar 05 '23 13:03 Pandapip1