回 4-4 分野 4 Issue と PR とレビュー
LGTM は
読み終えたらレビューの
この頁の流れ
LGTM の
LGTM は 「私は 良いと思う」
LGTM は
大事なのは
レビューでは
LGTM は どこから 来て、 何を 約束するか
2006 年の 記事に 残る出どころ
LGTM の
出典 2006 年の記事の 1 文
When the developer receives
a response of “looks good to
me,” or looks good to me
(lgtm) for short, they could
proceed to checkin.checkin は、Acked-by:) に
Gerrit の 点数の名前
オープンソースの
| 点 | 名前 (Gerrit の | 意味 |
|---|---|---|
| +2 | Looks good to me, approved | 良いと |
| +1 | Looks good to me, but someone else must approve | 良いと |
| 0 | No score | 点を付けない |
| −1 | I would prefer this is not submitted as is | このままでは |
| −2 | This shall not be submitted | 入れてはいけない |
Gerrit の
一方、
Gerrit の
GitHub の 3 つの返事
GitHub では、
| 返事 | GitHub の | 意味 |
|---|---|---|
| Comment | Leaves general feedback without explicitly approving or requesting changes. | 承認も |
| Approve | Signals that the changes are ready to merge. | 入れてよいという承認 |
| Request changes | Flags feedback that the author should address before merging. | 入れる |
LGTM と
LGTM With Comments
Google が
- 書いた
人が、 残りの 指摘を きちんと 直すと 信じられる - 残りの
指摘は、 直さなくても よい物である - 指摘が
小さい (import の 並べ替え、 近くの 打ち間違い、 使っていない 依存を 消す など)
時差の
レビューの略語の表
レビューで毎日出てくる
| 略語 | 元の英語 | 意味 | 重さ |
|---|---|---|---|
| LGTM | looks good to me | 私は | 承認 |
| SGTM | sounds good to me | (聞いた | 案への |
| PTAL | please take a look | 見てください。 | 頼む側の言葉 |
| WIP | work in progress | 作業中。 | 入れない印 |
| nit | nitpick | 細かい指摘 | 止めない |
| IMO | in my opinion | 私の意見では | 意見。 |
| FYI | for your information | 参考までに | 止めない |
| AFAIK | as far as I know | 私の | 確かでない |
| TL;DR | too long; didn't read | 長い文の | 説明の型 |
nit は
Google の Nit、 Optional、FYI
Google の
出典 Google の手引きの 3 つの印
Nit: This is a minor thing.
Technically you should do it,
but it won’t hugely impact
things.
Optional (or Consider): I
think this may be a good
idea, but it’s not strictly
required.
FYI: I don’t expect you to
do this in this CL, but you
may find this interesting to
think about for the future.日本語に
Nit:は、小さな こと。 直すのが 望ましいが、 大きな 影響は無い Optional:(またはConsider:) は、良い 案だと 思うが、 必ずではない FYI:は、この 変更で 直さなくて よい。 先の ために 知っておくと 役に立つ話
Conventional Comments の形
コメントの<label> [decorations]: <subject> で、
- label は
praise (ほめる )、 nitpick、 suggestion (提案)、 issue (問題)、 todo、 question、 thought、 chore、 note など - decorations の
(blocking)は承認を止める、 (non-blocking)は止めない、 (if-minor)は直しが 小さい ときだけ直せばよい - nitpick と
thought は 性質として 止めない物、 note は いつも 止めない 物と、 文書に 書かれている
動かして確かめる
1 本目は、
動かす 略語を見つけて元の英語を出す
▶ 実行ボタンを押すと結果がここに表示されます
実行すると、次の通りに出ます
LGTM, thanks!
LGTM looks good to me
nit: extra blank line.
NIT nitpick
Done. PTAL
PTAL please take a look
IMO this needs a test.
IMO in my opinionnit は
2 本目は、
動かす コメントの頭の印を読む
▶ 実行ボタンを押すと結果がここに表示されます
実行すると、次の通りに出ます
nitpick []
止めない
question ['non-blocking']
止めない
issue ['ux', 'non-blocking']
止めない
suggestion ['security']
チームの決まり
形が違う LGTMsuggestion (security) には、
3 本目は、[Draft]、Draft:、(Draft) の
動かす 題の頭の下書きの印を見分ける
▶ 実行ボタンを押すと結果がここに表示されます
実行すると、次の通りに出ます
True Draft: Add login form
True [Draft] Add login form
True (Draft) Add login form
False WIP: Add login form
False Add login formWIP も
略語は どこで外れるか
コメントの LGTM は 承認として 数えられない
GitHub で
LGTM は、 動かして 確かめた 印ではない
Gerrit が
例 確かめたことを足す
LGTM. I ran the migration
locally and it works.承認の あとに 直すと、 承認が 消える ことがある
GitHub の
nit を 全部 「直せ」と 読んでしまう
印のOptional: が
仕事で どう使うか
英語のレビューでよく
例 承認するとき
LGTM, thanks!
LGTM with a couple of nits.
LGTM once CI passes.with a couple of nits は
例 直したあと、もう一度頼むとき
Thanks! Addressed all
comments. PTAL.
Could you take another look?例 指摘を柔らかくするとき
nit: extra blank line.
Optional: consider using a
dict here.
What do you think about
splitting this function?指摘は、
よくある問い
LGTM は どう 読みますか
エル・ジー・ティー・エムと、
小文字の lgtm でも いいですか
どちらも
LGTM と 言われたら、 どう 返しますか
Thanks! や
SGTM と LGTM は どう 違いますか
SGTM は
LGTM の 画像や、 記事の LGTM は何ですか
PR の
確認クイズ
答えだと
1 問目
LGTM の
2 問目
GitHub で、
3 問目
Google の
4 問目
この
print(is_draft("WIP: Fix typo"))説明課題
GitHub で、レビューの人が
自分の
この回の 先の一歩
レビューの
もっと深く
この
一次資料
どれも 2026年10月30日に開いて、
- Google Mondrian (Niall Kennedy、2006 年 11 月、英語)。Mondrian より前のレビューは e-mail で
行い、 looks good to me (lgtm) の 返事で 変更を 入れられたこと - Review Labels (Gerrit の
文書、英語) 。Code-Review の+2 Looks good to me, approved と +1 Looks good to me, but someone else must approve、 +2 が 最後の 判断を 持つ人の ための 点である こと、 既定では +2 で 入れられる こと、 元の 意味、 Verified の 欄が 別にあること - Code Reviews (Chromium の
文書、英語) 。Code-Review +1 はLGTM とも 呼ぶこと、 コミッターの 変更の レビューの +1 は ほかの コミッター 1 人で足りる こと、触るディレクトリごとに OWNERS の レビューが要ること - Speed of Code Reviews (Google eng-practices、英語)。LGTM With Comments の
条件と、 LGTM の意味 - How to write code review comments (Google eng-practices、英語)。Nit、
Optional (Consider)、 FYI の 印と、 印を 付ける理由 - About pull request reviews (GitHub の
文書、英語) 。Comment、Approve、 Request changes の説明 - Approving a pull request with required reviews (GitHub の
文書、英語) 。承認の必須と 古い レビューを 外す設定が あると、 コードを 変える push で 承認が 外れること - About pull requests (GitHub の
文書、英語) 。下書きのPR は 入れられず、 コードの 持ち主に レビューの 依頼が 自動で 飛ばないこと - Draft merge requests (GitLab の
文書、英語) 。題の頭の[Draft]、Draft:、(Draft)で下書きになること - Conventional Comments (英語)。label と
decorations の 形、 blocking と non-blocking、 文書の 例のコメント - Submitting patches (Linux カーネルの
文書、英語) 。looks good to me の返事を Acked-by の 行に 置き換える ことがあること - Wiktionary の LGTM、SGTM、WIP、tl;dr、PTAL、nitpick、IMO、AFAIK、FYI (英語)。
略語の 元の 英語と読み方 - Qiita の
「いいね」が 。記事の「LGTM」に 変わります (Qiita Blog、2020 年 3 月) 評価の ボタンの 名前を 替えたこと
次へ
分野 4 Issue と PR とレビューの回
- 4-1コミットメッセージの英語命令形の
題、 Conventional Commits準備中 - 4-2Issue を
英語で書く再現の 手順、 expected と actual準備中 - 4-3PR の
説明を 英語で書くwhat と why、 テストしたこと準備中 - 4-4LGTM と
レビューの略語LGTM、 WIP、 PTAL、 nit、 IMO、FYIこの回