diff options
author | Android Partner Docs <noreply@android.com> | 2017-10-11 14:15:38 -0700 |
---|---|---|
committer | Clay Murphy <claym@google.com> | 2017-10-11 19:19:53 -0700 |
commit | 640dc96c73017d3bd2e2c507adcff02046ccfd3f (patch) | |
tree | 25296dfcb38dae95ee022015e64e78bbe646ff9c /en/source/view-patches.html | |
parent | 4ede9c52630a6da877e7ca9c1c3801a34ddb7179 (diff) | |
download | source.android.com-640dc96c73017d3bd2e2c507adcff02046ccfd3f.tar.gz |
Docs: Changes to source.android.com
- 171873304 Update documentation for types-only minor version package. by Android Partner Docs <noreply@android.com>
- 171849317 Update CTS/CTS-Verifier downloads for CTS-Oct-2017 Releases by Android Partner Docs <noreply@android.com>
- 171727609 Updating panic button to 8.0 by hvm <hvm@google.com>
- 171724554 Fixing typo by hvm <hvm@google.com>
- 171604727 Removed gerund form to match TOC bar on left. by cqn <cqn@google.com>
- 171576699 Small editorial changes to make content more skimmable. by cqn <cqn@google.com>
- 171546973 Add Clang toolchain to home News section by claym <claym@google.com>
- 171540129 Corrected a typo from "has" to "have" in CTS public setup... by Android Partner Docs <noreply@android.com>
- 171354973 Clean-up edits for consistency on the HCI Requirements do... by cqn <cqn@google.com>
- 171242784 Explain Clang is lone supported toolchain going forward by claym <claym@google.com>
- 171198827 Added researcher acknowledgement by Android Partner Docs <noreply@android.com>
- 171178686 Added researcher acknowledgement by Android Partner Docs <noreply@android.com>
- 171169962 Added missing " to jit-workflow.png by daroberts <daroberts@google.com>
- 171167568 Announce KASAN+KCOV on the home page by daroberts <daroberts@google.com>
- 171093616 Updated Researcher acknowledgement by Android Partner Docs <noreply@android.com>
- 171086588 Add Building a Pixel kernel with KASAN +KCOV by daroberts <daroberts@google.com>
- 171084549 Move CVE-2017-0710 to Google devices section by Android Partner Docs <noreply@android.com>
- 171063850 Removed duped content that is now in /source/view-patches... by cqn <cqn@google.com>
- 171061697 Create a /source/view-patches page so that the nav does n... by cqn <cqn@google.com>
- 171050148 Update home page with October 2017 security release by daroberts <daroberts@google.com>
- 171028732 Devsite localized content from translation request 5cdc34... by Android Partner Docs <noreply@android.com>
- 170937091 Add tags for October security backport releases (these do... by Android Partner Docs <noreply@android.com>
- 170911440 Remove CVE-2017-0605 from bulletin by Android Partner Docs <noreply@android.com>
- 170883975 Add AOSP links to the Oct 2017 Android Security bulletin by daroberts <daroberts@google.com>
- 170883918 Add AOSP links to Oct 2017 Pixel bulletin by daroberts <daroberts@google.com>
PiperOrigin-RevId: 171873304
Change-Id: I51cdbbbf00bdf43374c06638a8db4f8e87dbcdf7
Diffstat (limited to 'en/source/view-patches.html')
-rw-r--r-- | en/source/view-patches.html | 142 |
1 files changed, 142 insertions, 0 deletions
diff --git a/en/source/view-patches.html b/en/source/view-patches.html new file mode 100644 index 00000000..6b69f3cc --- /dev/null +++ b/en/source/view-patches.html @@ -0,0 +1,142 @@ +<html devsite> + <head> + <title>View Patches</title> + <meta name="project_path" value="/_project.yaml" /> + <meta name="book_path" value="/_book.yaml" /> + </head> + <body> + <!-- + Copyright 2017 The Android Open Source Project + + Licensed under the Apache License, Version 2.0 (the "License"); + you may not use this file except in compliance with the License. + You may obtain a copy of the License at + + http://www.apache.org/licenses/LICENSE-2.0 + + Unless required by applicable law or agreed to in writing, software + distributed under the License is distributed on an "AS IS" BASIS, + WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. + See the License for the specific language governing permissions and + limitations under the License. + --> + + <p> + If you want to view all patches to the Android Open Source Project, or + if you are reviewing or verifying a change, look in the <a + href="https://www.google.com/url?sa=D&q=https%3A%2F%2Fandroid-review.googlesource.com%2F%23%2Fq%2Fstatus%3Aopen%2BOR%2Bstatus%3Aclosed"> + AOSP Gerrit</a>. For more information on how to find a specific change, see <a + href="https://gerrit-review.googlesource.com/Documentation/user-search.html"> + Gerrit Code Review - Searching Changes</a>. + </p> + + <h2 id="reviewing-a-change">Reviewing a change</h2> + + <p> + If you are assigned to be the <em>Reviewer</em> for a change, you need + to determine the following: + </p> + + <ul> + <li>Does this change fit within this project's stated purpose?</li> + <li>Is this change valid within the project's existing architecture? + </li> + <li>Does this change introduce design flaws that will cause problems in + the future?</li> + <li>Does this change follow the best practices that have been + established for this project?</li> + <li>Is this change a good way to perform the described function?</li> + <li>Does this change introduce any security or instability risks?</li> + </ul> + + <p> + If you approve of the change, mark it with LGTM ("Looks Good to Me") + within Gerrit. + </p> + + <h2 id="verifying-a-change">Verifying a change</h2> + + <p> + If you are assigned to be the <em>Verifier</em> for a change, you need + to do the following: + </p> + + <ul> + <li>Patch the change into your local client using one of the Download + commands.</li> + <li>Build and test the change.</li> + <li>Within Gerrit select the <strong>Reply</strong> button. This + brings up a comment box where you can mark the change as + <em>Verified</em> or not, and add a message explaining what problems + were identified.</li> + </ul> + + <h2 id="downloading-changes-from-gerrit">Downloading changes from Gerrit + </h2> + + <p> + A submission that has been verified and merged will be downloaded with + the next <code>repo sync</code>. If you wish to download a specific + change that has not yet been approved, run + </p> + + + <pre + class="devsite-terminal devsite-click-to-copy"> +repo download <var>TARGET CHANGE</var></pre> + + <p> where <code><var>TARGET</var></code> is the local directory into + which the change should be downloaded and + <code><var>CHANGE</var></code> is the change number as listed in + Gerrit. For more information, see the <a + href="https://source.android.com/source/using-repo.html">Repo reference + </a>. + </p> + + <h2 id="how-do-i-become-a-verifier-or-reviewer">How do I become a Verifier + or Reviewer?</h2> + + <p> + In short, contribute high-quality code to one or more of the Android + projects. For details about the different roles in the Android Open + Source community and who plays them, see <a + href="https://source.android.com/source/roles.html">Project Roles</a>. + </p> + + <h2 id="diffs-and-comments">Diffs and comments</h2> + + <p> + To open the details of the change within Gerrit, click on the <em>Id + number</em> or <em>Subject</em> of a change. To compare the + established code with the updated code, click the file name under + <em>Side-by-side diffs</em>. + </p> + + <h2 id="adding-comments">Adding comments</h2> + + <p> + 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. + </p> + + <p> + 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 + <strong>Save</strong>, only you can see your comment. + </p> + + <p> + 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. + </p> + + </body> + +</html> |