Bug 152665 - Too easy to add contributors to wrong place in contributors.json
Summary: Too easy to add contributors to wrong place in contributors.json
Status: RESOLVED DUPLICATE of bug 163183
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: Other
Hardware: All All
: P2 Minor
Assignee: Michael Catanzaro
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-01-03 09:16 PST by Michael Catanzaro
Modified: 2018-01-11 07:33 PST (History)
9 users (show)

See Also:


Attachments
Patch (234.34 KB, patch)
2016-01-03 09:21 PST, Michael Catanzaro
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
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 ***