- About Scala
- Documentation
- Code Examples
- Software
- Scala Developers
community reviewing
Thu, 2010-01-07, 11:32
Dear all,
Before the holidays we started doing code reviews for all commits totrunk. We use the "review board" software on http://chara2.epfl.ch/
A post-commit script creates a review request and assigns it to theuser specified in the commit message ("review by username").
Now here's the thing: we have a special user named "community". If the commiter considers his changes as moderately critical, hecan assign the review request to this user.
The idea is that everybody in the community who's knowledgeable or interested in the compiler / library sources can perform the reviewof such a commit.
The password of the "community" user matches its username. To perform a review just log in, pick a review request, examine thechanges ("View Diff" button), then do the review ("Review" button).If you think the changes are fine, just tick the "Ship It" box when reviewing.
You can also add comments to any change by clicking the line number.
Cheers: Lukas
Before the holidays we started doing code reviews for all commits totrunk. We use the "review board" software on http://chara2.epfl.ch/
A post-commit script creates a review request and assigns it to theuser specified in the commit message ("review by username").
Now here's the thing: we have a special user named "community". If the commiter considers his changes as moderately critical, hecan assign the review request to this user.
The idea is that everybody in the community who's knowledgeable or interested in the compiler / library sources can perform the reviewof such a commit.
The password of the "community" user matches its username. To perform a review just log in, pick a review request, examine thechanges ("View Diff" button), then do the review ("Review" button).If you think the changes are fine, just tick the "Ship It" box when reviewing.
You can also add comments to any change by clicking the line number.
Cheers: Lukas
On Thu, Jan 7, 2010 at 5:32 AM, Lukas Rytz <lukas.rytz@epfl.ch> wrote: