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
chore: update AUTHORS list #34694
Conversation
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> |
There was a problem hiding this comment.
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?
There was a problem hiding this comment.
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?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, add it here
Line 414 in dd0c522
Ujjwal Sharma <usharma1998@gmail.com> <ryzokuken@disroot.org> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cool 👍
There was a problem hiding this comment.
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.
This could be automated by an Action. Any takers? |
@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. |
@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> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Another duplicate
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done 👍
There was a problem hiding this comment.
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
Okay @mmarchini Will remove the duplicates. |
There was a problem hiding this 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:
- Adding the entries to
.mailmap
- 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> |
There was a problem hiding this comment.
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> |
There was a problem hiding this comment.
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> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Duplicate.
I'm closing this as it was superseded by #35280. |
Checklist
make -j4 test
(UNIX), orvcbuild test
(Windows) passes