LoginSignup
0
1

More than 5 years have passed since last update.

プルリクエストを出してレビューしてもらった時に指摘されたことメモ

Last updated at Posted at 2017-05-24

GitHubを使い始めて2週間目です。
プルリクエストを出してレビューしてもらう際に同じことを二度と言われないようにするべく、レビュー時に指摘されたことを逐一書き足して行きます。

経験豊富な方から見て、「これも気をつけるべし」というのがあればバシバシご指摘ください。

指摘されたことメモ

  • 不要なファイルは消す
  • 余分な空行は消す(式中も、最下部も)
  • ルーティングにresourcesを使う場合は不要なものは省く
  • ビューを変更した際はスクショを貼り付ける
  • GET '○○/○○' よりは、プレフィックスで指定した方が良い(redirect設定時)
  • デバッグ時の記述は消す(binding.pryとか)(作業の難易度が高いほど、見直しを忘れがちになるので注意)

今後も随時書き足して行きます。

0
1
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
1