Skip to content
This repository has been archived by the owner on Feb 9, 2023. It is now read-only.

Ensure that author preamble has handle or email #44

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

axic
Copy link
Member

@axic axic commented Feb 15, 2022

Fixes #39.

ethereum/EIPs#4560 was merged, so this can be merged too.

@MicahZoltu
Copy link
Contributor

I think we want to allow some authors to have no contact details. Maybe we should rename the field to "editors" (though that is confusing with "EIP editors") to better indicate "these are the people who can automatically make changes" rather than "these are the people involved in authoring this EIP".

@axic
Copy link
Member Author

axic commented Feb 16, 2022

Maybe we should rename the field to "editors"

I had a discussion about that here: ethereum/EIPs#797

@MicahZoltu
Copy link
Contributor

It would seem that I was even involved in that discussion. If you want to reopen it and change to maintainer you'll have my support. Would require a bit of legwork to update author everywhere and replace it with maintainers everywhere (including bot, website, all historic EIPs), but if someone wants to put in that effort I support the change.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Enforce authors have a username/email
3 participants