Pull Requestを送る前に確認したいこと

1. コード整形

以下を実行して、Pull Requestに反映してください。

bundle exec rubocop -a

コーディングはインデントの仕方をとっても様々な流派があり、プロジェクトの中で書き方が統一されていないと非常に見辛くなってしまいます。そこで、プロジェクトのルールを決め、ツールを使ってコード整形をするのが一般的です。

BCDiceではRuboCopを使ってコード整形をしています。上記コマンドを実行すると、RuboCopがルール違反を見つけ、直せる部分は自動で修正してくれます。

• RuboCop is 何？ - Qiita

2. テスト

以下を実行して、エラーが発生しないか確認してください。

bundle exec rake test

BCDiceでは、コードの修正によりダイスボットの挙動が変わっていないか確認するためにテストを用意しています。上記コマンドを実行すると、テストを通過するの確認が行われます。これに通過しない場合、何か意図しない変更をしたことになるので、コードの修正が必要になります。