This page describes the WebM Project code review process.
To be the Approver for a change, you need to determine the following:
Depending on the branch this change will be committed to, some of these criteria are relaxed. See the repository layout page for more information on our branch policy.
If you approve of the change, you will then mark it with LGTM (“Looks Good to Me”) within Gerrit.
To verify a change, you should:
To open the details of the change within Gerrit, click on the Id number or Subject of a change. To compare the established code with the updated code, click the file name under Side-by-side diffs.
Anyone in the community can use Gerrit to add inline comments to code submissions. A good comment will be relevant to the line or section of code to which it is attached in Gerrit. It might be a short and constructive suggestion about how a line of code could be improved, or it might be an explanation from the author about why the code makes sense the way it is.
To add an inline comment, double-click the relevant line of the code and write your comment in the text box that opens. When you click Save, only you can see your comment.
To publish your comments so that others using Gerrit will be able to see them, click the Publish Comments button. Your comments will be emailed to all relevant parties for this change, including the change owner, the patch set uploader (if different from the owner), and all current reviewers.