Skip to content

Instantly share code, notes, and snippets.

@gurezo
Last active April 21, 2023 14:12
Show Gist options
  • Star 1 You must be signed in to star a gist
  • Fork 0 You must be signed in to fork a gist
  • Save gurezo/bc414240a6a3f8f26f9dda42f47d5f60 to your computer and use it in GitHub Desktop.
Save gurezo/bc414240a6a3f8f26f9dda42f47d5f60 to your computer and use it in GitHub Desktop.

PR コメントプリフィックス

[MUST]: 絶対直してほしいなというときに付けます。強い指摘になるので、言葉遣いには少し気をつけます
[MUST] 通信エラーのハンドリングが不足しているので処理の追加をお願いします!
[MUST] 不要な変更がコミットされているので取り消してほしいです!
[IMO]: 自分ならこうするかな〜、自分はこう思うのだけどどうだろう?というときに
  • In my opinionの略
[IMO] `何かしらのコード` みたいに書いたほうがスッキリすると思います!
[IMO] この処理はxxxというクラスに移したほうが責務的には良さそうです。

[NITS]: 細かい指摘や、軽い修正をして欲しいときに(nits pick)

[NITS] ここのコメント、不要なら消しちゃっても良さそう。
[NITS] `console.log` 残ったままになってる 👀

[Q]:ここ質問したい!という時に付けます。

  • Questionの略
  • 実装で不明瞭だった点や、そもそも自分の知らないことについて聞いたりします。
[Q] ここの処理は非同期で実行される可能性はありますか?
[Q] xxxって機能(書き方)があるんですね、具体的にどういうものなのでしょうか?

引用
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment