-
Notifications
You must be signed in to change notification settings - Fork 5.2k
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
Add protocol for author list changes #5129
Conversation
Hi! I'm a bot, and I wanted to automerge your PR, but couldn't because of the following issue(s): (fail) eip-1.md
|
I would like to see a more compelling argument for why this is necessary than "because other publications do it". My default position is that the author list follows the same rules as any other changes to the EIP and doesn't need to be called out as a special case. |
@@ -203,6 +203,10 @@ It occasionally becomes necessary to transfer ownership of EIPs to a new champio | |||
|
|||
If you are interested in assuming ownership of an EIP, send a message asking to take over, addressed to both the original author and the EIP editor. If the original author doesn't respond to the email in a timely manner, the EIP editor will make a unilateral decision (it's not like such decisions can't be reversed :)). | |||
|
|||
## Changes to the Author List | |||
|
|||
Changes to the author list of an EIP in Draft require either approval from either an editor or every author listed in either the old or new author list. Changes to EIPs in Review or Last Call require approval from both an editor and authors, unless the authors cannot be contacted. Changes to the author list of a Stagnant EIP follow the rules for the new status. Changes to the author list of a Final EIP are not allowed. Changes to the author list of a Living EIP only require editor approval. |
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.
Changes to the author list of an EIP in Draft require either approval from either an editor or every author listed
Too many "either" in this sentence.
If I'm understanding this correctly this would be an increase from any author approving to all authors approving. Why increase the threshold?
the old or new author list
So a PR with no old authors and ten new authors would be merged if all ten new authors approved without any old authors approving?
Overall I think these rules are complex, and I don't see what problem they're solving. Maybe you could explain a bit more why this is necessary?
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.
Honestly, I don't know what rule they are meant to solve. Now that I think of it, it's probably actually a legal thing to prevent libels and/or misattribution. I'll close this.
Most publications have a protocol for handling author list changes. This PR adds a protocol for this type of change.