masyuko0222

@machida 確認とれました。お忙しいところありがとうございました！

@machida 度々すみません。 ブラウザサイズによって、イベントが無いときの「近日開催イベント」表示の仕方が違うかもしれません。こちらお間違えなかったでしょうか👀 以下は、Event・RegularEvent共にdestroy_allをした後の画面になります。 (環境依存でしたらすみません) #### 横1366 縦728  #### 横758 縦700  イベントがない場合は、後者のように「◯◯開催のイベントはありません」とウィンドウサイズに限らず、表示したいと思っていました。 お手数ですがご確認よろしくお願いいたします。

@machida お忙しいところ失礼します。 `/regular_events`ページだと表示が下図のようになってしまいますので、お時間ある際に修正していただけますと幸いです。  `/events`ページは問題ありません。

@machida 確認がとれましたー！お忙しいところ何度もありがとうございました！ @junohm410 ご都合よろしければこちらのレビューをお願いしたいです。 忙しかったら、遠慮なく仰ってくださいー🙇

@junohm410 レビューありがとうございました！再レビューよろしくお願いいたします！ - コードは全部ご指摘通りの修正をしました。それぞれ返信しています。 - 冒頭にあった変更確認手順についても修正しました。これは自分が勘違いしていました、お手数おかけしてすみません。 また、コミットメッセージについてもありがとうございます。 リファクタリングのコミットは、「rubocopの修正」レベルの粒度で考えちゃっていました・・・。 リファクタリングにも自分の意図や意志が入ってくるので、ちゃんとメッセージを書いてあげた方がいいですね。以降気を付けます。

@junohm410 参考記事を読んだ結果、やっぱりデフォルト値はなくしました😅 シンプルな引数が一番可読性が高そうです。 度々恐縮ですが、修正をしたので再レビューいただいてもいいでしょうかm(__)m

@junohm410 何度も失礼しました！ありがとうございます！ @komagata Approveされましたので、レビューのほどお願いいたしますー。

@komagata リファクタリングし切れてないので、レビューはまだ大丈夫です🙇‍♂️ １点だけgitについてのご相談で、`pull --rebase`と`push --force-with-lease`をしたあとの、Github上での履歴が心配です。 ### 現状 切り貼りしてますが、以下のようなコミットとレビューコメントの流れになっています。  前のコミット→コメント→新しいコミット、の順番です。 ### `pull --rebase`と`push --force-with-lease`後  コメント→前のコミット+新しいコミット、になってしまう気がします。 (今までやってきた感じ) これが自分の操作が悪いのかが判断がつかず困っており、避けられるなら操作方法が知りたいです・・・！

@komagata > githubのコメント（コミットコメントではなく）の位置関係などは気にしなくていいと思います。 承知しました、一番気になってた箇所はここでした！ありがとうございます！

@komagata 度々すみませんーFatControllerをリファクタリングする作業は、別ブランチを切ってもよろしいでしょうか？ また、その際のレビューもkomagataさんで大丈夫でしょうかー？