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
Contributor

@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 <[email protected]>
Mateusz Krawczuk <[email protected]>
Jonathan MERCIER <[email protected]>
Ujjwal Sharma <[email protected]>
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
Contributor 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

.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Cool 👍

Copy link
Contributor 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
Contributor 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 <[email protected]>
Hachimi Aa (Sfeir) <[email protected]>
Daniel Estiven Rico Posada <[email protected]>
Ruben Bridgewater <[email protected]>
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
Contributor 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
Contributor Author

Okay @mmarchini Will remove the duplicates.

@yashLadha
Copy link
Contributor 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 <[email protected]>
Saajan <[email protected]>
Yash Ladha <[email protected]>
Xu Meng <[email protected]>
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 <[email protected]>
Ben Bucksch <[email protected]>
Eli Schwartz <[email protected]>
Ash Cripps <[email protected]>
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 <[email protected]>
Francisco Ryan Tolmasky I <[email protected]>
Ye-hyoung Kang <[email protected]>
Myles Borins <[email protected]>
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.

7 participants