-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
meta-issue: should RFC header include author name and/or contact info? #449
Comments
This information is already included in the git commit, though. |
@steveklabnik Maybe I did not write the first paragraph of my description clearly enough. Look at the recent commits. Such as commit bc5b034 or commit 336ccaa . Perhaps more pertinently, look at the text associated with each of those commits:
In each of those cases, the author according to the git metadata is the person who merged the RFC. At best you might be able to go back through the history of the merge to discover that the branches originated from niko and nick, respectively, not brian and aaron. I attempted to preserve the author metadata when I merged RFC #378. But its not necessarily part of everyone else's workflow. And in any case, if we ever have a broad revision of the RFC directory structure on par with what I did in PR #367, then we again lose the metadata. |
I guess yes, if people mess with the metadata, it's not reliable then. The merge is |
withdrawing suggestion; (leaving out an author better reflects the community-driven nature of the process). |
I have often relied on github metadata to identify immediately the RFC author. But this does not work without some amount of effort on the part of people maintaining the RFC repository to try to preserve author identification in the commits.
Should we add an
Author: "Your Name here" <email@domain.here>
to the RFC header? (And I guess give people the option of deleting the line from their RFC draft if they do not want to be personally identified with the RFC forever, though I am not sure whether trying to accommodate that request makes sense...)The text was updated successfully, but these errors were encountered: