Exercises5 6 4

[yutokyokutyo]

Rails Tutorial Exercises5_6_4

http://www.railstutorial.org/book/filling_in_the_layout#sec-layout_exercises

内容

Helperについてのテストを追加しました。 spec/supportにhelperを記述していましたが、includeすることによって記述しないで参照できるようにしました。 動作を確認したところGreenになりました。 MasterにMergeしたいと思っております。 レビューをお願いいたします。

動作確認

@tacahilo @kitak @gs3 @keokent

[hfm]

dbaf778 のコミットは違うexerciseだから余計かな． このPRをmergeする前に， #7 を片付けよう．

[hfm]

LGTM

[keokent]

このプルリクのコミット２つを見ると https://github.com/yutokyokutyo/sample_app/commit/dbaf7783f0ee138c2df0018b2ded27d1b1954630 これと https://github.com/yutokyokutyo/sample_app/commit/bfdc71514395560a5ff1d6e3181c29112b46f12c これなんだけど、本当だったら Exercises5_6_3 でやったhttps://github.com/yutokyokutyo/sample_app/commit/197f5fb93d5edb7eee2b957864d19a12d9a4dd1c のコミットが 上２つの間にないといけないんじゃないかな。

予想なんだけど、局長は1つ前のプルリクの Exercise5_6_3 のブランチで https://github.com/yutokyokutyo/sample_app/commit/dbaf7783f0ee138c2df0018b2ded27d1b1954630 のコミットのあとにこのプルリクの課題をやるために git checkout -b Exercises5_6_4 ってしたんだと思います。 そのあと1つ前の課題の Exercise5_6_3 でhttps://github.com/yutokyokutyo/sample_app/commit/197f5fb93d5edb7eee2b957864d19a12d9a4dd1c のコミットを追加して master にマージしたので、 Exercises5_6_4のブランチさんはhttps://github.com/yutokyokutyo/sample_app/commit/197f5fb93d5edb7eee2b957864d19a12d9a4dd1c のコミットのことなんて知らないし、Exercise5_6_3 が masterにマージされたこと（マージコミット）も知らないので、差分として https://github.com/yutokyokutyo/sample_app/commit/dbaf7783f0ee138c2df0018b2ded27d1b1954630 のコミットが表示されているのだと思います。

なので、このExercises5_6_4のブランチに master の内容を反映させてあげれば良いと思います。

反映のさせ方については、ちょっと考えてみよう :octocat:

【追記】 今日 1日 MTG等で席を外していて見るの遅くなってしまったけど、Exercises5_6_1 〜 Exercises5_6_3 のプルリク確認しました！良いと思います！ :hamster:

[yutokyokutyo]

@tacahilo @keokent レビューありがとうございます！ けおけんさんのレビューの通りでした！ ぐっさんさんにも今日またお聞きして納得して解決することができました。 無事MasterにMergeすることができそうなのでMergeします。