Development/gerrit/PatchReview/ja
TDF LibreOffice Document Liberation Project Community Blogs Weblate Nextcloud Redmine Ask LibreOffice Donate
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-Review
と Verified
について
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システムが正常にビルドできない限り、パッチはマージされません。