【Exercises9_6_5】Viewについての重複するコードをパーシャルを用いて排除し、リファクタリングしました。

[yutokyokutyo]

Rails Tutorial 【Exercises9_6_5】

内容

• [x] new.html.erbとedit.html.erbについて、重複するコードをパーシャルを用いて排除し、リファクタリングしました。
• [x] Exercisesにて、サインアップ時のフォーム名をConfirmationからConfirm Passwordに変更したので、その変更に適応したテストに修正しました。

  テスト結果

全体テストでGreenを確認しました。

% bundle exec rspec spec/                                                                                                                                                                    (git)-[Exercises9_6_5]
...........................................................................................

Finished in 3.84 seconds
91 examples, 0 failures

Randomized with seed 18617

演習内容

http://www.railstutorial.org/book/updating_and_deleting_users#sec-updating_deleting_exercises

---

この内容でMasterにマージしたいと思います。 レビューをお願いいたします。

@tacahilo @kitak @gs3 @keokent

[keokent]

~swp というファイルが含まれちゃってるね。 こういうのは git add . する前に git status して確認したり、 git commit するときに 変更を加えたファイル一覧を見て確認するようにしよう :octocat:

[yutokyokutyo]

指摘いただいた改行ミスとインデントの修正をしました！ Materにマージしたい思います！ レビューありがとうございました！:ram:

[yutokyokutyo]

@keokent

~swp というファイルが含まれちゃってるね。 こういうのは git add . する前に git status して確認したり、 git commit するときに 変更を加えたファイル一覧を見て確認するようにしよう :octocat:

staus をどんどん打って確認する癖をつけていきたいと思います！ ありがとうございました！

[keokent]

このプルリクマージしちゃってるけど、本来なら ~swp のような不要なファイルは削除しておくべきだったかなぁ（レビューで注意したつもりだけど、ちゃんと伝わってなかったかも・・・）

暇な時にこっそり削除するでもいいし、次回からは 不要なファイルは含めない ことを徹底しよう :octocat:

[hfm]

余計なファイルをコミットしてしまってるよ，と指摘を受けた時はちゃんと消すようにしましょう :sunflower: git rm でもいいし，rmしてからgit add -Aでもいいし．

[yutokyokutyo]

@keokent @tacahilo

ああああ。不注意でした(>_<) ご指摘いただいたのにすみません。 二度とこのようなことのないように努めます。

[keokent]

:+1: