-
Notifications
You must be signed in to change notification settings - Fork 29
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
Doublons in contributors list #32
Comments
Transferred to the repo for the thanks website |
The names need to be deduplicated in https://github.com/rust-lang/rust/blob/master/.mailmap. |
Thanks, and sorry that I am of so little help, I kind of understand the principle of this file and I could complete it but then I don't have access to the history of email of the different authors (I can access only the last AFAIK). And it seems some pages were manually corrected already without any change in this file: no more duplication in https://thanks.rust-lang.org/rust/1.51.0/ (and I could not find entries in the file for the unduplicated authors). |
What needs to be fixed?
First, I'm not too proud of this, this is very stupid glitch, but that's it. I was just curious about rust contributors list
(like this one : https://thanks.rust-lang.org/rust/1.51.0/)
I was looking for French names and I found some relatively unusual French-like names several times like "Amanieu d'Antras" which appear at positions 61 and 88.
This can be found using this script:
which answers:
Page(s) Affected
Nearly all on thanks.rust-lang.org but definitively:
https://thanks.rust-lang.org/rust/all-time
https://thanks.rust-lang.org/rust/1.51.0/
https://thanks.rust-lang.org/rust/1.50.0/
https://thanks.rust-lang.org/rust/1.49.0/
https://thanks.rust-lang.org/rust/1.48.0/
https://thanks.rust-lang.org/rust/1.47.0/
Suggested Improvement
Merge the homonymous lines numbers.
The text was updated successfully, but these errors were encountered: