Skip to content
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

AUTHORS #331

Merged
merged 1 commit into from
Sep 10, 2013
Merged

AUTHORS #331

merged 1 commit into from
Sep 10, 2013

Conversation

zpao
Copy link
Member

@zpao zpao commented Sep 10, 2013

Created a .mailmap file with all of the associations, then used
git + perl to create the AUTHORS file. In theory these should all get
picked up by npm.

I used ABC order so it would remain unbiased and automatable. I wish we
could go back and fill out the history or at least fix the commits we
have from CommitSyncScript, but oh well.

git shortlog -se | perl -spe 's/^\s+\d+\s+//' > AUTHORS

@zpao
Copy link
Member Author

zpao commented Sep 10, 2013

It's been pointed out that leaving CommitSyncScript in the authors file is silly. I agree, so will fix.

@sophiebits
Copy link
Collaborator

👍 lgtm

Created a .mailmap file with all of the associations, then used
git + perl to create the AUTHORS file. In theory these should all get
picked up by npm.

I used ABC order so it would remain unbiased and automatable. I wish we
could go back and fill out the history or at least fix the commits we
have from CommitSyncScript, but oh well.

This also includes the script I used to automate this process in the
future.
@zpao zpao merged commit 63b58cf into facebook:master Sep 10, 2013
bvaughn pushed a commit to bvaughn/react that referenced this pull request Aug 13, 2019
[Flare] Update event component displayName
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants