フィヨルドブートキャンプというプログラミングスクールで初めてのPRに対するレビューをたくさんやっています。これらの指摘は世界中で行われていると思うので無駄が減るように書きました。 Files changedを確認したか 真っ先に注意したい点。 ローカルの自分のエディタ上で確認していたとしても、PRを作ったら必ずFiles changedを確認する癖をつけよう。GitHubのソースコードビューワーの機能で色々と気づける点もある。 明らかにFiles changedを一回も見てないなというPRはレビュワーもそれと分かります。 lintを通しているか rubyだったらrubocop、jsだったらeslint どういうルールが良いかという高度は話題は置いておいて、まずはlintを通しているか否か。 ファイルの末尾に改行があるか これ どのエディターでも設定で自動的に入るようにできるので設定すること