tools,meta: make AUTHORS reflect all the contributors #88
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* Add new `update-authors.js` tool that traverses commit history using `git log` and creates authors list. * Decouple common functions into a module shared between tools. * Add `.mailmap` file with proper names and emails of active collaborators. This file is automatically parsed by Git so that `git log --format='%aN <%aE>'` won't show duplicate entries in case something was committed with misconfigured Git. E.g., without this file @tshemsedinov appears twice in the list, as Timur Shemsedinov and as tshemsedinov, but with mailmap applied Git automatically coalesces these entries to use the full name only.
aqrln
commented
Feb 26, 2017
AUTHORS
Outdated
| @@ -1,3 +1,6 @@ | |||
| Timur Shemsedinov <timur.shemsedinov@gmail.com> | |||
| Dimon Durak <Dimon Durak> | |||
There was a problem hiding this comment.
@dimon-durak can you please provide your real name and email if you want to be included to the list?
There was a problem hiding this comment.
Dmitry Borisov
dimon.durak@gmail.com
There was a problem hiding this comment.
Thanks, I've added an entry for you in .mailmap and regenerated the list of contributors.
aqrln
commented
Feb 26, 2017
AUTHORS
Outdated
| Alexey Orlenko <eaglexrlnk@gmail.com> | ||
| Mykola Bilochub <nbelochub@gmail.com> | ||
| Vlad <Dzyubavlad@gmail.com> |
There was a problem hiding this comment.
@DzyubSpirit please check if you email is correct here. And how would you prefer your full name to be spelled?
There was a problem hiding this comment.
Email is correct and I prefer to be Dzyuba WhatIsLove.
Thanks.
There was a problem hiding this comment.
@DzyubSpirit I think we prefer real names unless you have strong reasons to not be called Vlad Dzyuba. I was asking about the correct spelling of your last name, actually :)
What about email, maybe I should have been more explicit too: the question is whether it should be written with a capital letter or I can lowercase it.
There was a problem hiding this comment.
@aqrln, okey. Right spelling of my last name is Dziuba. Let it be this way. My email case does not matter, so you can do what it is needed with it.
* Don't use a Set together with an array, O(n^2) instead of O(n*log(n)) is fine for such tool but the code is shorter. * Use a regular for-of loop instead of reduce.
New commit message for 8915cd7: tools: generate authors list automatically * Add new `update-authors.js` tool that traverses commit history using `git log` and creates authors list. * Decouple common functions into a module shared between tools. * Add `.mailmap` file. This file is automatically parsed by Git so that `git log --format='%aN <%aE>'` won't show duplicate entries in case something was committed with misconfigured Git. E.g., without this file @tshemsedinov appears twice in the list, as Timur Shemsedinov and as tshemsedinov, but with mailmap applied Git automatically coalesces these entries to use the full name only. This file now includes the active collaborators (@tshemsedinov for a reason and @aqrln and @belochub just in case) and proper names and emails for two contributors who committed with invalid ones.
|
ping @belochub @tshemsedinov |
|
@tshemsedinov ping |
aqrln
added a commit
that referenced
this pull request
Mar 13, 2017
* Add new `update-authors.js` tool that traverses commit history using `git log` and creates authors list. * Decouple common functions into a module shared between tools. * Add `.mailmap` file. This file is automatically parsed by Git so that `git log --format='%aN <%aE>'` won't show duplicate entries in case something was committed with misconfigured Git. E.g., without this file @tshemsedinov appears twice in the list, as Timur Shemsedinov and as tshemsedinov, but with mailmap applied Git automatically coalesces these entries to use the full name only. This file now includes the active collaborators (@tshemsedinov for a reason and @aqrln and @belochub just in case) and proper names and emails for two contributors who committed with invalid ones should be placed there. PR-URL: #88
aqrln
added a commit
that referenced
this pull request
Mar 13, 2017
aqrln
added a commit
that referenced
this pull request
Mar 13, 2017
* Add new `update-authors.js` tool that traverses commit history using `git log` and creates authors list. * Decouple common functions into a module shared between tools. * Add `.mailmap` file. This file is automatically parsed by Git so that `git log --format='%aN <%aE>'` won't show duplicate entries in case something was committed with misconfigured Git. E.g., without this file @tshemsedinov appears twice in the list, as Timur Shemsedinov and as tshemsedinov, but with mailmap applied Git automatically coalesces these entries to use the full name only. This file now includes the active collaborators (@tshemsedinov for a reason and @aqrln and @belochub just in case), proper names and emails of contributors who committed with invalid ones should be placed here. PR-URL: #88
aqrln
added a commit
that referenced
this pull request
Mar 13, 2017
* Add all the contributors using `tools/update-authors.js`. * Sort the mail map alphabetically. PR-URL: #88
aqrln
added a commit
that referenced
this pull request
Mar 13, 2017
aqrln
added a commit
that referenced
this pull request
Mar 13, 2017
* Add new `update-authors.js` tool that traverses commit history using `git log` and creates authors list. * Decouple common functions into a module shared between tools. * Add `.mailmap` file. This file is automatically parsed by Git so that `git log --format='%aN <%aE>'` won't show duplicate entries in case something was committed with misconfigured Git. E.g., without this file @tshemsedinov appears twice in the list, as Timur Shemsedinov and as tshemsedinov, but with mailmap applied Git automatically coalesces these entries to use the full name only. This file now includes the active collaborators (@tshemsedinov for a reason and @aqrln and @belochub just in case), proper names and emails of contributors who committed with invalid ones should be placed here. PR-URL: #88
aqrln
added a commit
that referenced
this pull request
Mar 13, 2017
* Add all the contributors using `tools/update-authors.js`. * Sort the mail map alphabetically. PR-URL: #88
aqrln
added a commit
that referenced
this pull request
Mar 13, 2017
Merged
aqrln
added a commit
that referenced
this pull request
Mar 13, 2017
* Add new `update-authors.js` tool that traverses commit history using `git log` and creates authors list. * Decouple common functions into a module shared between tools. * Add `.mailmap` file. This file is automatically parsed by Git so that `git log --format='%aN <%aE>'` won't show duplicate entries in case something was committed with misconfigured Git. E.g., without this file @tshemsedinov appears twice in the list, as Timur Shemsedinov and as tshemsedinov, but with mailmap applied Git automatically coalesces these entries to use the full name only. This file now includes the active collaborators (@tshemsedinov for a reason and @aqrln and @belochub just in case), proper names and emails of contributors who committed with invalid ones should be placed here. PR-URL: #88
aqrln
added a commit
that referenced
this pull request
Mar 13, 2017
* Add all the contributors using `tools/update-authors.js`. * Sort the mail map alphabetically. PR-URL: #88
aqrln
added a commit
that referenced
this pull request
Mar 13, 2017
aqrln
added a commit
that referenced
this pull request
Apr 2, 2017
* Add all the contributors using `tools/update-authors.js`. * Sort the mail map alphabetically. PR-URL: #88
aqrln
added a commit
that referenced
this pull request
Apr 2, 2017
* Add all the contributors using `tools/update-authors.js`. * Sort the mail map alphabetically. Backport-of: #88
belochub
pushed a commit
that referenced
this pull request
Jan 22, 2018
* Add new `update-authors.js` tool that traverses commit history using `git log` and creates authors list. * Decouple common functions into a module shared between tools. * Add `.mailmap` file. This file is automatically parsed by Git so that `git log --format='%aN <%aE>'` won't show duplicate entries in case something was committed with misconfigured Git. E.g., without this file @tshemsedinov appears twice in the list, as Timur Shemsedinov and as tshemsedinov, but with mailmap applied Git automatically coalesces these entries to use the full name only. This file now includes the active collaborators (@tshemsedinov for a reason and @aqrln and @belochub just in case), proper names and emails of contributors who committed with invalid ones should be placed here. PR-URL: #88
belochub
pushed a commit
that referenced
this pull request
Jan 22, 2018
* Add all the contributors using `tools/update-authors.js`. * Sort the mail map alphabetically. PR-URL: #88
belochub
pushed a commit
that referenced
this pull request
Jan 22, 2018
belochub
pushed a commit
that referenced
this pull request
Jan 22, 2018
* Add new `update-authors.js` tool that traverses commit history using `git log` and creates authors list. * Decouple common functions into a module shared between tools. * Add `.mailmap` file. This file is automatically parsed by Git so that `git log --format='%aN <%aE>'` won't show duplicate entries in case something was committed with misconfigured Git. E.g., without this file @tshemsedinov appears twice in the list, as Timur Shemsedinov and as tshemsedinov, but with mailmap applied Git automatically coalesces these entries to use the full name only. This file now includes the active collaborators (@tshemsedinov for a reason and @aqrln and @belochub just in case), proper names and emails of contributors who committed with invalid ones should be placed here. PR-URL: #88
belochub
pushed a commit
that referenced
this pull request
Jan 22, 2018
* Add all the contributors using `tools/update-authors.js`. * Sort the mail map alphabetically. PR-URL: #88
belochub
pushed a commit
that referenced
this pull request
Jan 22, 2018
Merged
belochub
pushed a commit
to metarhia/mdsf
that referenced
this pull request
Jul 19, 2018
* Add new `update-authors.js` tool that traverses commit history using `git log` and creates authors list. * Decouple common functions into a module shared between tools. * Add `.mailmap` file. This file is automatically parsed by Git so that `git log --format='%aN <%aE>'` won't show duplicate entries in case something was committed with misconfigured Git. E.g., without this file @tshemsedinov appears twice in the list, as Timur Shemsedinov and as tshemsedinov, but with mailmap applied Git automatically coalesces these entries to use the full name only. This file now includes the active collaborators (@tshemsedinov for a reason and @aqrln and @belochub just in case), proper names and emails of contributors who committed with invalid ones should be placed here. PR-URL: metarhia/jstp#88
belochub
pushed a commit
to metarhia/mdsf
that referenced
this pull request
Jul 19, 2018
* Add all the contributors using `tools/update-authors.js`. * Sort the mail map alphabetically. PR-URL: metarhia/jstp#88
belochub
pushed a commit
to metarhia/mdsf
that referenced
this pull request
Jul 19, 2018
belochub
pushed a commit
to metarhia/mdsf
that referenced
this pull request
Jul 19, 2018
* Add new `update-authors.js` tool that traverses commit history using `git log` and creates authors list. * Decouple common functions into a module shared between tools. * Add `.mailmap` file. This file is automatically parsed by Git so that `git log --format='%aN <%aE>'` won't show duplicate entries in case something was committed with misconfigured Git. E.g., without this file @tshemsedinov appears twice in the list, as Timur Shemsedinov and as tshemsedinov, but with mailmap applied Git automatically coalesces these entries to use the full name only. This file now includes the active collaborators (@tshemsedinov for a reason and @aqrln and @belochub just in case), proper names and emails of contributors who committed with invalid ones should be placed here. PR-URL: metarhia/jstp#88
belochub
pushed a commit
to metarhia/mdsf
that referenced
this pull request
Jul 19, 2018
* Add all the contributors using `tools/update-authors.js`. * Sort the mail map alphabetically. PR-URL: metarhia/jstp#88
belochub
pushed a commit
to metarhia/mdsf
that referenced
this pull request
Jul 19, 2018
belochub
pushed a commit
to metarhia/mdsf
that referenced
this pull request
Jul 21, 2018
* Add new `update-authors.js` tool that traverses commit history using `git log` and creates authors list. * Decouple common functions into a module shared between tools. * Add `.mailmap` file. This file is automatically parsed by Git so that `git log --format='%aN <%aE>'` won't show duplicate entries in case something was committed with misconfigured Git. E.g., without this file @tshemsedinov appears twice in the list, as Timur Shemsedinov and as tshemsedinov, but with mailmap applied Git automatically coalesces these entries to use the full name only. This file now includes the active collaborators (@tshemsedinov for a reason and @aqrln and @belochub just in case), proper names and emails of contributors who committed with invalid ones should be placed here. PR-URL: metarhia/jstp#88
belochub
pushed a commit
to metarhia/mdsf
that referenced
this pull request
Jul 21, 2018
* Add all the contributors using `tools/update-authors.js`. * Sort the mail map alphabetically. PR-URL: metarhia/jstp#88
belochub
pushed a commit
to metarhia/mdsf
that referenced
this pull request
Jul 21, 2018
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Right now we only have those who have made valuable contributions noted in the AUTHORS file. But there are people who have made minor contributions, like fixing typos, updating the links and such, to examples and README.md (which is effectively a part of documentation, and documentation is a part of the software, according to the MIT license). And however small their contributions may be, these are still contributions and it is fair to add those people to the AUTHORS file.
Additionally, the process of updating this file is now automated.
Changes:
update-authors.jstool that traverses commit history usinggit logand creates authors list..mailmapfile with proper names and emails of active collaborators. This file is automatically parsed by Git so thatgit log --format='%aN <%aE>'won't show duplicate entries in case something was committed with misconfigured Git. E.g., without this file @tshemsedinov appears twice in the list, as Timur Shemsedinov and as tshemsedinov, but with mailmap applied Git automatically coalesces these entries to use the full name only.tools/update-authors.js.