peps icon indicating copy to clipboard operation
peps copied to clipboard

Allow authors to use different emails in different PEPs?

Open hugovk opened this issue 1 year ago • 7 comments

Dear fellow @python/pep-editors,

This came up in https://github.com/python/peps/pull/3618, the build failed because the PEP author has previously used his personal email address in other PEPs, but his work email in this new PEP:

  • https://github.com/python/peps/pull/3618#issuecomment-1889480882

Is it a hard requirement that authors must use the same email for all PEPs?

Or can we allow authors to use whichever they find most appropriate for each PEP?


Code-wise, we'd need at least need to modify _verify_email_addresses:

  • not to raise if more than one email
  • and in the PEP 0 table of names & emails:
    • print just one email
    • print all emails
    • or even ditch that table altogether

https://github.com/python/peps/blob/2cbb99c5776a18a518ab90518174be7291a9228b/pep_sphinx_extensions/pep_zero_generator/writer.py#L311-L341

hugovk avatar Jan 12 '24 15:01 hugovk

The philosophical issue is what is the email for. Are there two authors who happen to have the same name but are distinguished by email? Or does the email convey a role? Or is it simply a way to reach the author? What should happen if the author changes jobs or retires?

gvanrossum avatar Jan 12 '24 16:01 gvanrossum

I'm minded to suggest keeping the email check, as it means that the reader can be confident that the authors are in fact the same person, rather than two people that happen to have the same name. As a by-product it means that if someone updates their email address, it must be updated everywhere, which a reviewer might forget to check.

A

AA-Turner avatar Jan 16 '24 21:01 AA-Turner

what is the email for

We can have a useful talk about this. Anybody is welcome to such a talk and to this conversation.

Julian-Dumitrascu avatar Jan 17 '24 08:01 Julian-Dumitrascu

Does this need more deliberation? Or should we just assume that the status quo wins, and there is little appetite for change?

gvanrossum avatar Jan 18 '24 18:01 gvanrossum

I'd recommend updating the email check logic to allow the same person to have more than one email address in different PEPs. It is normal for anyone to own more than one email address anyway, and it is the author's choice. We can list the author's multiple email addresses on PEP 0. We shouldn't need to update past PEPs with new email address, unless the author wants to do it.

Mariatta avatar Jan 18 '24 18:01 Mariatta

@gvanrossum We can think about some change.

Julian-Dumitrascu avatar Jan 18 '24 18:01 Julian-Dumitrascu

I'd recommend updating the email check logic to allow the same person to have more than one email address in different PEPs. It is normal for anyone to own more than one email address anyway, and it is the author's choice. We can list the author's multiple email addresses on PEP 0. We shouldn't need to update past PEPs with new email address, unless the author wants to do it.

If we were talking about different forums (e.g. two different Discourse servers) I agree users need freedom in which email they use. But given that PEPs are a flat namespace, what purpose is served by allowing multiple email addresses? If someone submits one PEP as a private user and another as an engineer at some company, the most likely outcome to me seems that some years later one or the other email address becomes invalid. This can be annoying when (as has happened) we want to contact a user year later with a question if an ambiguity in a PEP needs to be resolved.

Note that in the issue linked from the top comment here, the affected user writes

Using my personal email here is not a significant problem -- the ToB email would accurately reflect that this is for work rather than just funsies, but we have it tracked on our side anyways 🙂

so I don't think this case is strong enough to change the restriction.

gvanrossum avatar Jan 18 '24 22:01 gvanrossum

Thanks all for the comments, let's leave it as is. We can always reconsider later.

hugovk avatar May 28 '24 06:05 hugovk

:+1:

Rosuav avatar May 28 '24 06:05 Rosuav