Bug 152665

Summary: Too easy to add contributors to wrong place in contributors.json
Product: WebKit Reporter: Michael Catanzaro <mcatanzaro>
Component: Tools / TestsAssignee: Michael Catanzaro <mcatanzaro>
Status: RESOLVED DUPLICATE    
Severity: Minor CC: annulen, ap, commit-queue, fred.wang, glenn, lforschler, mcatanzaro, ossy, slomo
Priority: P2    
Version: Other   
Hardware: All   
OS: All   
Attachments:
Description Flags
Patch none

Description Michael Catanzaro 2016-01-03 09:16:50 PST
contributors.json has three lists: committers, contributors (who are neither committers nor reviewers; good to make it easy to CC people or bots in Bugzilla), and reviewers. But the file is very long, so it's easy to not realize this and add a name to the wrong list, or get confused about which part of the file you are in.

For instance, it took three tries to get Sebastian's name in the right place:

http://trac.webkit.org/changeset/190959
http://trac.webkit.org/changeset/194432
http://trac.webkit.org/changeset/194511

I also noticed Frédéric was in the wrong place:

http://trac.webkit.org/changeset/194512

I only caught these two cases because these are folks whom I happen to know, and I was looking through the file while moving myself to the reviewers list. There are probably many other contributors in the wrong place.

Make it more difficult to add people in the wrong place by splitting these lists out into three separate files.
Comment 1 Michael Catanzaro 2016-01-03 09:21:35 PST
Created attachment 268139 [details]
Patch
Comment 2 Alexey Proskuryakov 2016-01-03 09:50:08 PST
Comment on attachment 268139 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=268139&action=review

> Tools/ChangeLog:3
> +        Too easy to add contributors to wrong place in contributors.json

I think that this breaks Bugzilla autocomplete, as well as a contributors page on webkit.org.
Comment 3 Michael Catanzaro 2016-01-03 10:55:24 PST
(In reply to comment #2)
> I think that this breaks Bugzilla autocomplete, as well as a contributors
> page on webkit.org.

You're right. I updated the Python code that processes these files, but neither Bugzilla nor the contributors page use that code. It seems like the code I updated was only used by the WebKit commit bot.
Comment 4 Konstantin Tokarev 2018-01-11 07:33:18 PST
I think this change is not relevant anymore after r206973. contributors.json is a flat alphabetically sorted list now.

*** This bug has been marked as a duplicate of bug 163183 ***