Development/gerrit/PatchReview/ja

    From The Document Foundation Wiki
    This page is a translated version of the page Development/gerrit/PatchReview and the translation is 52% complete.
    Outdated translations are marked like this.


    After you completed the gerrit setup, and you or others have submitted patches, you can start to review patches on gerrit.

    Finding patches waiting for review

    ./logerrit query status:open

    will show you all patches waiting for review. see gerrit documentation for all the fancy things you can do in addition (for scripting etc.).

    Getting a patch

    ./logerrit cherry-pick CHANGEID

    with CHANGEID being the Change-Id shown by the output of the gerrit query

    Updating patch

    If you found and fixed a problem during review. You might update the existing patch in gerrit the following way:

    git status; git diff

    which makes sure that you have only the single cherry-picked commit and the extra changes on top

    git commit -a --amend

    which updates the last commit locally

    ./logerrit submit

    which pushes all uncommited changes into gerrit; it updates the existing patch in gerrit if the ID stayed the same in the commit message

    Comment and Approve

    If you want to add comments or approve the patch, the best way is to do it from the Gerrit web interface. Comment by clicking a line of a changed file and when you are done click the Reply... link to publish your review.

    Alternatively, you might just push the patch but it will not add the nice comments about reviewing.

    Code-ReviewVerified について

    Gerritでパッチを見ると、以下の2つのステータスコードがあります:

    • Code-Review
    • Verified

    査読者とCIシステム(jenkins)、さらに潜在的にはあなたはこれらのフィールドをパッチの状況のために使うことになります。

    Code-Review

    −2, −1, 0, +1, +2が割り当てられます

    −2は作者が使用し、「作業中」であることを示すためのものです。-2ではパッチがマージされず、−2をつけた人のみがパッチを削除できます。

    もし大きなパッチを作業するのであれば、パッチのアップロードは大いに歓迎されますので、−2にマークして、すべてのプラットフォームで正常にビルドできることを確認してください。

    −1は査読者が使用し、パッチの変更が必要であることを示します。

    0 はコメントをつける時に使用し、パッチのマージの可否に影響しません。

    +1 は査読者が使用し、パッチはOKだが、査読者に代替案があることを示します。

    +2 は作者が使用し、査読不要であることを示します(これはコアな開発者のみが行うことができ、また注意深く使用されるべきです)。 あなたのパッチをマージする人は、マージ前に+2を使用するでしょう。+2のみがマージ可能であるためです。+2に設定できるかどうかは査読者のpush権限に依存します。この権限は個人個人に与えられます。

    注意:−1が付いていたり、−2で応答がない限り、パッチはマージされません。

    Verified

    −1、0、+1 が割り当てられます

    −1はCIシステムがビルドに失敗したことを示します(注意:必ずしもあなたのパッチに問題があるわけではなく、masterが壊れていることもあります)

    −1は査読者が使用し、期待された結果が再現できないことを示します。

    0はコメントをつける時に使用し、パッチのマージの可否に影響しません。

    +1はCIシステムがビルドに成功したことを示します。

    +1 は査読者が使用し、期待された結果が再現できたことを示します。

    注意:CIシステムが正常にビルドできない限り、パッチはマージされません。

    Note pin.svg

    Note:
    For more information on those labels, see the gerrit documentation. For gerrit configuration relevant to our projects, see Access Control Lists for all dev-related repositories, for core, and for all other projects.