Mercurial > pylearn
changeset 1209:5ff1d375fc33
added a first list of code review system.
| author | Frederic Bastien <nouiz@nouiz.org> |
|---|---|
| date | Tue, 21 Sep 2010 16:27:47 -0400 |
| parents | 0186805a93e7 |
| children | cbe1fb32686c |
| files | doc/v2_planning/code_review.txt doc/v2_planning/committees.txt doc/v2_planning/index.txt |
| diffstat | 3 files changed, 18 insertions(+), 1 deletions(-) [+] |
line wrap: on
line diff
--- /dev/null Thu Jan 01 00:00:00 1970 +0000 +++ b/doc/v2_planning/code_review.txt Tue Sep 21 16:27:47 2010 -0400 @@ -0,0 +1,16 @@ + +Code Review +=========== + +Some system that we should check: + +- `rietveld <http://code.google.com/p/rietveld/>` Made by Guido van Rossum +- `Gerrit <http://code.google.com/p/gerrit/>` +- `track PeerReviewPlugin <http://trac-hacks.org/wiki/PeerReviewPlugin>` Could be integrated with the current ticket system? + +- `feature request at assembla <http://feedback.assembla.com/forums/5433-feature-requests/suggestions/253297-add-a-code-review-tool-e-g-reviewboard->` + +What we could want from our code review: + +- integrate with our ticket system? + - Should we keep our current ticket system?
--- a/doc/v2_planning/committees.txt Tue Sep 21 14:42:38 2010 -0400 +++ b/doc/v2_planning/committees.txt Tue Sep 21 16:27:47 2010 -0400 @@ -13,7 +13,7 @@ o Hooks: Pascal L.*, Ian G., Olivier B. o Existing: Fred B.* * Layers: Razvan P., Xavier G., Arnaud B., David W. F. -* social engineering, code review and incentives: Fred B.*, Pascal L., James B., Olivier D. +* social engineering, code review and incentives: Fred B.*, Pascal L., James B., Olivier D., Arnaud B. Issues to be tackled in the future: