0
0

Delete article

Deleted articles cannot be recovered.

Draft of this article would be also deleted.

Are you sure you want to delete this article?

プルリクのフィードバックでのNG例

Posted at

プルリクフィードバックのNG例

この変数は何を意味していますか?

プルリクで質問はダメ。

置き換えるとすれば、こう。

  • 省略しすぎなので、フルの単語でお願いします。
  • 〇〇と〇〇の両方の解釈があるのでこうしてください。
  • コーディング規約にしたがってください。

〇〇はしないでください

「ハードコーディングしないでください」ではなく、〇〇にパラメーターを集約してくださいなどという指示に変える。

そもそもプルリク以前のスタートで間違ってることもある

コーディング規約のすり合わせがチームでできてないならそれはプルリクでやるべきことではない。
プルリクはコミットファイルだけに限られたスコープでのコミュニケーションであり、それを超えるスコープはプルリク以外の場でやるべき。

プルリクへのフィードバックができるほどの腕前ではないなら、3往復くらい会話をしてくれ。

0
0
0

Register as a new user and use Qiita more conveniently

  1. You get articles that match your needs
  2. You can efficiently read back useful information
  3. You can use dark theme
What you can do with signing up
0
0

Delete article

Deleted articles cannot be recovered.

Draft of this article would be also deleted.

Are you sure you want to delete this article?