Closed
Bug 613653
Opened 14 years ago
Closed 14 years ago
update credits page and figure out way to keep it up-to-date
Categories
(Add-on SDK Graveyard :: Documentation, defect)
Add-on SDK Graveyard
Documentation
Tracking
(Not tracked)
RESOLVED
FIXED
1.0b1
People
(Reporter: myk, Assigned: adw)
References
Details
Attachments
(1 file)
876 bytes,
patch
|
myk
:
review+
|
Details | Diff | Splinter Review |
The credits page is woefully out-of-date. We need to bring it back up-to-date and then keep it that way or remove it, since an out-of-date page is worse than no page at all for people who should be on it but aren't.
At first glance, at least the following names are missing:
Will Bamberg
Irakli Gozalishvili
Alexandre Poirot
Piotr Zalewa
Justin Scott (fligtar)
Dave Townsend (mossop)
Zbigniew Braniecki (gandalf)
Zandr Milewski
Assignee | ||
Comment 1•14 years ago
|
||
GitHub has a contributors list:
https://github.com/mozilla/addon-sdk/contributors
Since that excludes people like Zandr who might not necessarily touch the repo yet contribute, it's not really sufficient to only point to it. I also think it's a bad idea to list "GitHub contributors" and "non-GitHub contributors" in separate places, e.g., by listing the latter in our credits file and including a link to the former in the same file, since that might be construed as valuing one differently than the other.
We could write a script that fetches contributors from GitHub, combines them with a list of non-GitHub contributors, and updates our own credits file. Running this script would become part of the release process. What do people think about that?
GitHub provides an API for getting contributors:
http://develop.github.com/p/repo.html
e.g.:
http://github.com/api/v2/yaml/repos/show/mozilla/addon-sdk/contributors
But it lists GitHub nicknames, often without email addresses and full names. A better way might be to parse the commit history, which does include full names and emails:
http://github.com/api/v2/yaml/commits/list/mozilla/addon-sdk/master
Assignee | ||
Comment 2•14 years ago
|
||
(In reply to comment #1)
> But it lists GitHub nicknames, often without email addresses and full names. A
> better way might be to parse the commit history, which does include full names
> and emails:
Oh, but it only lists "recent commits", 35 at most it looks like.
Assignee | ||
Comment 3•14 years ago
|
||
Of course, if the script is run from within a git clone on a machine with git installed, it could query the clone for contributors rather than asking GitHub. Duplicates ("Drew Willcoxon" and "Drew Willcoxon :adw") and weird names ("aza@aza-raskins-macbook-air.local") would need to be resolved.
http://www.google.com/search?q=git+list+contributors
Reporter | ||
Comment 4•14 years ago
|
||
Alternately, we could get committers to commit to updating the list whenever they push anyone else's patch.
Assignee | ||
Comment 5•14 years ago
|
||
They (we) have done a great job so far! If everyone is responsible for it then no one is. If it stays accurate at all, one person will end up doing it for everybody else, so we should either make it easy for that person or just remove the file entirely.
Assignee | ||
Comment 6•14 years ago
|
||
Anyhow... :) This includes the list above and the output of `git log --format='%aN' | sort -u`.
Attachment #492921 -
Flags: review?(myk)
Reporter | ||
Comment 7•14 years ago
|
||
Comment on attachment 492921 [details] [diff] [review]
patch
Looks good. Please also add Ayan Shah to the list when checking this in.
I have also added an item to the release checklist about updating the page so it becomes part of the release process <https://wiki.mozilla.org/Labs/Jetpack/Release#Update_Credits_Page>.
Attachment #492921 -
Flags: review?(myk) → review+
Assignee | ||
Comment 8•14 years ago
|
||
https://github.com/mozilla/addon-sdk/commit/a0a9245151dfd502e9510159ac333206f3a28ff9
https://github.com/mozilla/addon-sdk/commit/44873ebea8fedc43ae6520a55bf54615d9e7aafb
https://github.com/mozilla/addon-sdk/commit/26cf5b1c79113e4670c25cbc39409e668fee8a2d
Assignee: nobody → adw
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Target Milestone: -- → 0.10
Reporter | ||
Updated•14 years ago
|
Target Milestone: 0.10 → 1.0b1
You need to log in
before you can comment on or make changes to this bug.
Description
•