-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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
Multiple entries in CONTRIBUTORS.md #1909
Comments
Hi Chris, Thanks for notifying me. There's something wrong with the file. There are over 16k lines. Which doesn't seem right. I will look into this. In the meantime can we stop merging PRs until I fix this? Thanks! |
Ok, it's not just your entry that's duplicated. My entries are duplicated too. The first time it happened looks like 3 days ago. Between @robotnamedEmily and @StevenCopeland commits. There's also extra files in the repo that shouldn't be there or are in the wrong folder... What's confusing too is it looks like many contributors are committing straight into master without creating a PR for review and approval first. I didn't think this was possible unless you're a collaborator. It also looks like some entries may have been overwritten... If anyone has any suggestions, please let me know. The settings do not allow me to restrict user perms. Only add collaborators and Protect Branches. |
@AliceWonderland Hey Alice, normally, users shouldn't have write access to this repository unless they are collaborators like you mentioned, so this is indeed quite peculiar. If this issue persists, you might want to contact GitHub support. |
@AliceWonderland I was look at how many contributors there were yesterday(599), and now there are 600. And no PRs have been merged recently! |
@AliceWonderland |
@AliceWonderland and @christianheinrichs Duplicate Contributors Formatter solves all your problems |
@shashankKeshava In the related issue #2015 you can see that two people already proposed PRs to programmatically sort the CONTRIBUTORS.md file. I am not sure when the pull requests #2162 (Artur Aciole), #2163 (Artur Aciole), #2197 (Julius Breckel) and your PR #3002 (Shashank Keshava) are going to be reviewed, but I guess we should all wait for Alice's answer. |
Thanks @christianheinrichs, @kylesferrazza, @shashankKeshava ! As of Nov 17 the CONTRIBUTORS file has been cleaned and up on master. I created a script to remove dupes before I saw these proposals. Which can still be used. I now have to check out the sort and PRs and also resolve merge conflicts for those PRs before I can merge. We now have 488 entries instead of 3300. Which doesn't seem quite right given we have 2000+ merges. The bad merges may not only have created dupes but overwritten entries. Hopefully not too many! I will catch up with what is going on with the sort programs and respond back! Until then please continue to hold off on merges. Thanks again for all your help and contributions!! :) |
@AliceWonderland Glad you got this worked out! Always happy to help. |
HII can you please assign me this issue. |
Many entries in the CONTRIBUTORS.md file are duplicated for some reason.
My entry for example appears 4 times. This can either be fixed automatically with a script or manually. If every contributor were to fix this themselves, take a look at pull request #1908, where I removed 3 duplicated entries of my contribution.
The text was updated successfully, but these errors were encountered: