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

chore: update AUTHORS list #34694

Closed
wants to merge 3 commits into from

Conversation

yashLadha
Copy link
Member

@yashLadha yashLadha commented Aug 9, 2020

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • tests and/or benchmarks are included
  • documentation is changed or added
  • commit message follows commit guidelines

@nodejs-github-bot nodejs-github-bot added the meta Issues and PRs related to the general management of the project. label Aug 9, 2020
AUTHORS Outdated
ExE Boss <3889017+ExE-Boss@users.noreply.github.com>
Mateusz Krawczuk <krawczukmat@gmail.com>
Jonathan MERCIER <jmercier@cng.fr>
Ujjwal Sharma <ryzokuken@igalia.com>
Copy link
Member

@lpinca lpinca Aug 9, 2020

Choose a reason for hiding this comment

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

This is definitely a duplicate. Please remove it and update .mailmap. Did you check if there are more?

Copy link
Member Author

Choose a reason for hiding this comment

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

@lpinca This email is also not present in .mailmap should I update it in the .mailmap as well?

Copy link
Member

Choose a reason for hiding this comment

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

Yes, add it here

node/.mailmap

Line 414 in dd0c522

Ujjwal Sharma <usharma1998@gmail.com> <ryzokuken@disroot.org>
.

Copy link
Member Author

Choose a reason for hiding this comment

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

Cool 👍

Copy link
Member Author

Choose a reason for hiding this comment

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

I think the update-authors also need to change for this.

@mmarchini
Copy link
Contributor

This could be automated by an Action. Any takers?

@yashLadha
Copy link
Member Author

@mmarchini I just used the script in the tools, will try to update the script so that this case doesn't occur and update the AUTHORS file.

@mmarchini
Copy link
Contributor

@yashLadha might be better to do @lpinca suggestion above for this PR and then open another one for changes on the script.

AUTHORS Outdated
Alex R <alexei.rudenko@gmail.com>
Hachimi Aa (Sfeir) <hachimi.aa@sfeir.com>
Daniel Estiven Rico Posada <danielrico.posada@gmail.com>
Ruben Bridgewater <ruben.bridgewater@maibornwolff.de>
Copy link
Member

Choose a reason for hiding this comment

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

Another duplicate

Copy link
Member Author

Choose a reason for hiding this comment

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

Done 👍

Copy link
Member

Choose a reason for hiding this comment

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

You also need to add this email address to the .mailmap, otherwise it will show up again during the next update

@yashLadha
Copy link
Member Author

Okay @mmarchini Will remove the duplicates.

@yashLadha
Copy link
Member Author

@targos @lpinca Done requested changes.

Copy link
Member

@richardlau richardlau left a comment

Choose a reason for hiding this comment

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

More duplicates. These should be handled by:

  1. Adding the entries to .mailmap
  2. Rerunning the generator script

Bartlomiej Brzozowski <bart.brzozowski@janeasystems.com>
Saajan <saajan.sn@gmail.com>
Yash Ladha <yashladhapankajladha123@gmail.com>
Xu Meng <dmabupt@gmail.com>
Copy link
Member

Choose a reason for hiding this comment

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

Duplicate.

Kirill Shatskiy <shaackiy@gmail.com>
Ben Bucksch <ben.bucksch@beonex.com>
Eli Schwartz <eschwartz@archlinux.org>
Ash Cripps <ashley.cripps@ibm.com>
Copy link
Member

Choose a reason for hiding this comment

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

Duplicate.

Johannes Schöpp <schoepp@rz.uni-frankfurt.de>
Francisco Ryan Tolmasky I <tolmasky@gmail.com>
Ye-hyoung Kang <keepyourhonor@gmail.com>
Myles Borins <mylesborins@github.com>
Copy link
Member

Choose a reason for hiding this comment

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

Duplicate.

@lpinca
Copy link
Member

lpinca commented Oct 5, 2020

I'm closing this as it was superseded by #35280.

@lpinca lpinca closed this Oct 5, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
meta Issues and PRs related to the general management of the project.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants