SeaeeesSan/SimpleFolderIcon

三角眼鏡の指摘事項

dwl398 opened this issue · 1 comments

dwl398 commented

各項目でissueにしたりしなかったりするといいかも

image

  • if () の後の改行のルールを統一する 基本的にコードが横長になると編集がしにくい（横にスクロールが必要になると見にくい）ので改行がおすすめ

image

  • こういう書き方とかだと横に伸びなくてコードレビューもしやすい

image

  • privateつけたり付けなかったりを統一する

image

  • 機能追加とリファクタリングは同じコミットに含めない

    • リファクタリングしたい気持ちをグッと我慢してコミットを分ける リファクタリングした部分だけStageしないでコミットとかでもOK

  • あとinitのコミットに成果物を含めるんじゃ無い！！！！！！！！！！！！！！！

SeaeeesSan commented

ありがとうございます！