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

Refactor AUTHORS file to make it easier to add new AUTHORS with each release #9077

Closed
misterdjules opened this issue Jan 22, 2015 · 1 comment
Closed

Refactor AUTHORS file to make it easier to add new AUTHORS with each release #9077

misterdjules opened this issue Jan 22, 2015 · 1 comment
Assignees
@misterdjules
Labels
tools v0.12
Milestone
0.11.17

Comments

@misterdjules
Copy link

The current AUTHORS file is ordered by time of first contribution to the project, and there is actually no way to automatically get rid of duplicated entries for the same author with different email addresses.

The release tools should generate an AUTHORS file ordered alphabetically by authors' names and should generate a .mailmap file that maps authors' names to email addresses.
@misterdjules misterdjules added this to the 0.11.16 milestone Jan 22, 2015
@misterdjules misterdjules self-assigned this Jan 22, 2015
@misterdjules misterdjules modified the milestones: 0.11.17, 0.11.16 Feb 2, 2015
misterdjules pushed a commit to misterdjules/node that referenced this issue Feb 5, 2015
tools: improve tooling around AUTHORS file
387ac80 
Replace tools/updateAuthors.awk with tools/update-authors.js. The new
tool generates an AUTHORS file that is stable-ordered alphanumerically.

Fixes nodejs#9077.
misterdjules pushed a commit to misterdjules/node that referenced this issue Feb 5, 2015
tools: improve tooling around AUTHORS file
31610d0 
Replace tools/updateAuthors.awk with tools/update-authors.js. The new
tool generates an AUTHORS file that is stable-ordered alphanumerically.

Fixes nodejs#9077.

PR: nodejs#9088
PR-URL: nodejs#9088
Reviewed-By: Timothy J Fontaine <tjfontaine@gmail.com>
@misterdjules
Copy link
Author

Fixed by 31610d0, which actually doesn't generate a .mailmap file automatically. It was implemented, but it turns out that it's easy to automatically add bogus entries to it.

gibfahn pushed a commit to ibmruntimes/node that referenced this issue Oct 16, 2016
@evanlucas
2016-10-14, Version 6.8.1 (Current)
c6a397b 
* build: Fix building with shared zlib. (Bradley T. Hughes) [nodejs#9077](nodejs/node#9077)
* stream: fix `Writable` subclass instanceof checks (Anna Henningsen) [nodejs#9088](nodejs/node#9088)
* timers: fix regression with clearImmediate() (Brian White) [nodejs#9086](nodejs/node#9086)

PR-URL: nodejs/node#9104
kaiquewdev pushed a commit to kaiquewdev/node that referenced this issue Nov 26, 2016
@evanlucas
2016-10-14, Version 6.8.1 (Current)
113c697 
* build: Fix building with shared zlib. (Bradley T. Hughes) [nodejs#9077](nodejs/node#9077)
* stream: fix `Writable` subclass instanceof checks (Anna Henningsen) [nodejs#9088](nodejs/node#9088)
* timers: fix regression with clearImmediate() (Brian White) [nodejs#9086](nodejs/node#9086)

PR-URL: nodejs/node#9104
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@misterdjules misterdjules

Labels
tools v0.12
Projects
None yet
Milestone
0.11.17
Development

No branches or pull requests
1 participant
@misterdjules