search

yutokyokutyo / sample_app

Rails Tutorial 研修用
2 stars 0 forks source link

3.5 Exercises 2 #4

Closed yutokyokutyo closed 10 years ago

yutokyokutyo commented 10 years ago

3.5 Exercises 2

http://www.railstutorial.org/book/static_pages#sec-static_pages_exercises

spec/requests/static_pages_spec.rbについて、Let関数を使い重複を阻止しました。

MaterにMergeしたいのですが、レビューをお願いいたします。

動作確認

2014-07-09 18 16 41

@tacahilo @kitak @gs3 @keokent

hfm commented 10 years ago

s/Mater/Master/かな。細かいけど誤字は気をつけよう。

そしてテストは通りましたよー

$ bin/rake spec
/Users/usr0600296/.rbenv/versions/2.1.1/bin/ruby -S rspec ./spec/requests/static_pages_spec.rb
........

Finished in 0.12171 seconds
8 examples, 0 failures

Randomized with seed 37887
hiboma commented 10 years ago

:sushi:

hfm commented 10 years ago

@yutokyokutyo ところで、Exercises 2をやったということなんだけど、この課題をどう解釈したのかも説明があるとよいかな。

hfm commented 10 years ago

Let関数を使い重複を阻止しました。

「阻止」って言葉の使い方あってる？ ニュアンスは分かるから今のところ問題があるわけじゃあないけど、 なるべく正確な表現を心がけるのはコミュニケーションをする上で大切だよ ちなみに僕の場合は「解消」って言葉を使ったりするかな

hfm commented 10 years ago

あと、何が重複してるのかの説明がPR文に無いね このExerciseは、一体何が重複しているのか、重複の何が良くないのかを理解することがポイントだと思う

hfm commented 10 years ago

ヒント：DRY

keokent commented 10 years ago

spec/requests/static_pages_spec.rbについて、Let関数を使い重複を阻止しました。

という内容に対して、差分が多いような気がする。。

差分を見ると about や contact ページも作成している気がする。。

keokent commented 10 years ago

現状の master の コミット数が 4つだけなのは なんでなんだろう・・・

yutokyokutyo commented 10 years ago

@tacahilo

s/Mater/Master/かな。細かいけど誤字は気をつけよう。

Materになってましたね。次からは誤字がないかを確認してからPRしようと思います。

ところで、Exercises 2をやったということなんだけど、この課題をどう解釈してどう実装したのかも説明があるとよいかな。

なるほど。そうですね！ その説明があれば、自分で勘違いしたまま次に進むことがなくなりそうですね！ これからは、以下のように書くと伝わりますでしょうか？

「Ruby on Rails Tutorial Sample App」というタイトルが重複してしまい、DRYの考えに反してしまうため、Let関数(Rspecの重複を解消できる関数)を使い重複を解消しました。

具体的に自分で修正した点は以下のとおりです。

yutokyokutyo commented 10 years ago

@tacahilo @hidoma

そしてテストは通りましたよー

:sushi:

やったあ！それではMasterにマージします！

yutokyokutyo commented 10 years ago

「阻止」って言葉の使い方あってる？ ニュアンスは分かるから今のところ問題があるわけじゃあないけど、 なるべく正確な表現を心がけるのはコミュニケーションをする上で大切だよ ちなみに僕の場合は「解消」って言葉を使ったりするかな

そうですね。 日本語の使い方がたまにおかしくなってしまうので、もっと注意するようにします。 自分の文面を誰かに見てもらうという自覚を持ちます！ あと、これを読みます！

理科系の作文技術

Amazon: http://www.amazon.co.jp/dp/4121006240/

hfm commented 10 years ago

「Ruby on Rails Tutorial Sample App」というタイトルが重複してしまい、DRYの考えに反してしまうため、Let関数(Rspecの重複を解消できる関数)を使い重複を解消しました。 ...

いいと思う。 初めは丁寧に書いてみてもいいけど、毎回大量に文章書くのは大変だから、 「これぐらい書けば相手に伝わるかな」って見極めて、少ない文章に明確な情報を詰められるようにしてみてください :hamburger:

yutokyokutyo commented 10 years ago

@keokent

現状の master の コミット数が 4つだけなのは なんでなんだろう・・・

すいません。お昼ごはんを食べてたので返信が遅れてしまいました！

流れとしては、 もともと自分の「Exercises3_5_2」というブランチから「master」にマージがしたいというPRを出したかったのですが、最初に間違えてmasterにpushしてしまい、リモートの「Exercises3_5_2」ブランチと「master」ブランチの内容が同じになってしまうという問題が生じてしまいました。 そこで、リモートのmasterブランチにpushしてしまった変更を取り消すためにローカルでresetコマンドで前の状態に戻って、push -f コマンドにより強制的にpushしました。

おそらく、 git reset --soft HEAD^したときに、もとに戻りすぎてしまって、commit数が増えてしまったのかな？と思っています。 ううむ。

手順

  1. git reset --soft HEAD^
  2. git push -f origin HEAD^:master

詳細

https://gist.github.com/yutokyokutyo/0c423f8299366d2bddd5

https://gist.github.com/yutokyokutyo/b6cc89d37970497a2132

yutokyokutyo commented 10 years ago

@tacahilo ご査読ありがとうございます！

少ない文章に明確な情報を詰められるようにしてみてください :hamburger:

次はそこも意識して書いてみますね！ :baby_chick:

keokent commented 10 years ago

もともと自分の「Exercises3_5_2」というブランチから「master」にマージがしたいというPRを出したかったのですが、最初に間違えてmasterにpushしてしまい、リモートの「Exercises3_5_2」ブランチと「master」ブランチの内容が同じになってしまうという問題が生じてしまいました。

経緯把握しました！ 本来やりたかった内容自体は達成できていると思うので良いと思います :hamster: