インクリボンとタイプライター

セーブがしたいんです…

レビューの反省 2016年12月5日

仮として作った部分も消しておきましょう!
プルリクエストは、本番環境の機能を想定して作っていくものです。本番の環境で使わない部分の実装をプルリクエストで出すことはよくありません。
なので、ご自身で試されたり、仮のものとして書いたコードは全て消しておきましょう!
console.logやbinding.pryも上記と同じ理由で消すのが通例です!

 

deviseで生成されたファイルの中で、使っていないファイルは消してしまいましょう!

 

ルーティングは可能な箇所はresourcesを使って設定しましょう!

 

・scssにおいて、頻繁に出てくる色は変数にしてしまいましょう。

 

aタグではなく、link_to になおしましょう!

 

色などの変数は、stylesheets/config/variables.scssなどにまとめてしまって、application.scssにインポートしてあげたほうがいいですね!