プログラミング初心者はgit commitする前に必ずdiffを自分でレビューするクセを付けよう - give IT a try

プログラミング初心者向けのTipsです。 まあ、タイトルに書いたとおりなんですが、プログラミング初心者は（というか、プログラマならみんな）git commitする前にdiffを自分でチェックするようにしましょう。 それはなぜか？ しょーもないミスを自分で見つけるためです。 しょーもないミスというのは例えば、消し忘れのコメントや、デバッグ用に書き込んだprint文、無駄な空行、おかしなインデント、管理対象外とすべき一時ファイルや隠しファイル等々です。 def create @book = Book.new(book_params) puts @book.title # ほら、デバッグ用のputsが残ってるよ！！ if @book.save redirect_to @book, notice: '登録しました' else render :new # インデントが1文字ズレてるよ！！ end e

[wata_d]

見てるつもりでちょくちょくすり抜け、pushしてから気付くことが…orz

[tmatsuu]

せやな。せやせや。想定外の不要な変更をしてないか確認しような。git statusも確認するといいぞ。git commitは-mを指定せず-vをつけるようにするのもオススメじゃ。こちらからは以上です。

[taroidchang]

git

[sechs]

git add -p を使うと変更の塊ごとにインデックスに上げれるのでそこでセルフレビューになってる。この方法はなかなか良いと勝手に思ってる。

[turanukimaru]

これは大事なことだな。昨日のコードとか、なんでこのコード書いたんだっけ…？というコードになってたりするから意図が明確になるように書き直したりコメントを付けておくとか。

[takc923]

あとPR前にウェブUIで全体をセルフレビューするのもおすすめ

[ohesotori]

“ツールだけに頼るのはやめましょう。”gitkraken でいろいろやらかしてしまったので使うのやめたんだわ・・・orz