Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

update contributors file #668

Merged
merged 1 commit into from
Nov 28, 2018
Merged

update contributors file #668

merged 1 commit into from
Nov 28, 2018

Conversation

weissi
Copy link
Member

@weissi weissi commented Nov 27, 2018

Motivation:

Now, shortly before 2.0 is a good time to update the contributors list.

Modifications:

Ran the contributor generation script (ASCII'd my name)

Result:

All contributors get their proper credit.

Motivation:

Now, shortly before 2.0 is a good time to update the contributors list.

Modifications:

Ran the contributor generation script (ASCII'd my name)

Result:

All contributors get their proper credit.
@weissi weissi requested a review from Lukasa November 27, 2018 10:32
@@ -1,7 +1,7 @@
Tomer Doron <tomerd@apple.com> <tomer.doron@gmail.com>
Tomer Doron <tomerd@apple.com>
Max Moiseev <moiseev@apple.com>
Johannes Weiß <johannesweiss@apple.com>
Johannes Weiss <johannesweiss@apple.com> Johannes Weiß <johannesweiss@apple.com>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This change brought to you by HM Government.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

😂

@helje5
Copy link
Contributor

helje5 commented Nov 27, 2018

Why ASCII your name when we just got efficient UTF-8 strings?

@weissi
Copy link
Member Author

weissi commented Nov 27, 2018

Why ASCII your name when we just got efficient UTF-8 strings?

stuff breaks, such as git's .mailmap :(

@helje5
Copy link
Contributor

helje5 commented Nov 27, 2018

Doesn't that imply you have to ASCII me as well?

@weissi
Copy link
Member Author

weissi commented Nov 27, 2018

@helje5 so far we didn't .mailmap coalesce your name yet :). I have used both spellings in various places so want to coalesce them. You have been more consistent

@weissi weissi merged commit 59b3ac4 into apple:master Nov 28, 2018
@weissi weissi deleted the jw-uc branch November 28, 2018 09:53
@weissi weissi added the 🔨 semver/patch No public API change. label Nov 28, 2018
@weissi weissi added this to the 1.12.0 milestone Nov 28, 2018
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
🔨 semver/patch No public API change.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants