The list of contributors in committers.py should be a separate JSON

• Component: Tools / Tests
• Status: RESOLVED FIXED

[Ryosuke Niwa]

We should extract the list of contributors, committers, and reviewers into a JSON file.
There’s no point in forcing everyone to read through Python code.

[Ryosuke Niwa]

This will also allow Bugzilla and other websites and tools to parse the list of contributors in JavaScript more easily.

[Ryosuke Niwa]

Created attachment 198315 [details]
Cleanup

[Csaba Osztrogonác]

Comment on attachment 198315 [details]
Cleanup

LGTM, but using email/email and nick/nicks in the JSON is a little bit strange for me.
I prefer a simpler syntax, only email and nick. It is so easy to check if an instance
is a string or a list: isinstance(var, types.ListType) and isinstance(var, types.StringType)

[Csaba Osztrogonác]

Or we should recommend always using lists.

[Ryosuke Niwa]

(In reply to comment #4)
> Or we should recommend always using lists.

Let’s do that.

[Ryosuke Niwa]

Created attachment 198344 [details]
Always use emails and nicks

[Ryosuke Niwa]

Committed r148527: <http://trac.webkit.org/changeset/148527>

[Glenn Adams]

I notice that completion no longer works in BZ's CC, Assigned To etc, which may have to do with this patch.

[Ryosuke Niwa]

(In reply to comment #8)
> I notice that completion no longer works in BZ's CC, Assigned To etc, which may have to do with this patch.

Clear the cache.