Last change
on this file since 2495 was
2381,
checked in by achernya, 12 years ago
|
Merge r2380 from locker-dev to trunk
|
File size:
1.2 KB
|
Rev | Line | |
---|
[2380] | 1 | Scripts has a policy of formal code review on Zephyr. Commit messages |
---|
| 2 | are zephyred to -c scripts -i r[number], with the actual diff sent to |
---|
| 3 | -c scripts-auto -i commits. Commits are also emailed to |
---|
| 4 | scripts-commits@mit.edu. All commits should be reviewed by another |
---|
| 5 | scripts-team member before they are deployed. |
---|
| 6 | |
---|
| 7 | The following designations are in use, with the following meanings: |
---|
| 8 | "+1": I have reviewed and approve of this commit. |
---|
| 9 | |
---|
| 10 | "+0": I've glanced at the commit and it seems okay. |
---|
| 11 | |
---|
| 12 | "-0": I have mild issues with this commit, but am fine with it going |
---|
| 13 | in as-is and no further action being taken. |
---|
| 14 | |
---|
| 15 | "-1": I do not approve of this commit, it needs more work. This |
---|
| 16 | designation must include an explanation of the objection, and |
---|
| 17 | will likely result in further action being taken either by the |
---|
| 18 | committer or the reviewer to improve the commit. |
---|
| 19 | |
---|
| 20 | "-1, revert": I do not approve of this commit, and believe it to be |
---|
| 21 | actively harmful to the project, such as introducing a security |
---|
| 22 | hole. This designation must include an explanation of the |
---|
| 23 | objection. The committer should revert the commit. |
---|
| 24 | |
---|
| 25 | Reviewers should strive to give non-zero reviews whenever possible. |
---|
Note: See
TracBrowser
for help on using the repository browser.