OkinawaDevOps / okinawadevops.github.com

http://okinawadevops.github.io/
0 stars 0 forks source link

16 arakaki yuji #29

Closed Arakaki-Yuji closed 10 years ago

Arakaki-Yuji commented 10 years ago

もくもく会から数日過ぎましたが、もくもくした内容をまとめましたのでプルリクを送ります(^^*)

takumikinjo commented 10 years ago

:+1:

yamanetoshi commented 10 years ago

手元で動作確認してみたのですが

について対処した方が良くなると思いますが、如何でしょ。そのまま反映、でも良いと思っています。

takumikinjo commented 10 years ago

自環境では文字化けは見れませんでした。URLは、ハイパーリンクできたほうがいいかもですね確かに。この機にリンクだけ直されてはいかがでしょう。面白いかもしれません、プルリクエストの出し直しなるものをみんなでどう扱うか等。この場合、単にブランチで更新して再度プルリクエストでいいのですかね?やったことないためwktk

Arakaki-Yuji commented 10 years ago

レビューありがとうございます。 指摘していただいた ・文字化け ・リンク ・コマンド引用部分 は修正して再度プルリクエストを送らせて頂きます。(たぶんまた夜になると思いますが。。)

プルリクエストの出し直しは、kinjoさんの提案して頂いた通りブランチで更新して再度プルリクエストで良いかな?と思っております。

yamanetoshi commented 10 years ago

修正版はそのまま push すればこの pull request に反映されるはずですよー

16-Arakaki-Yuji branch に修正をもりこんで再度 push という意味です

確か再度作り直さなくても良かったはずですので試してみて下さい。

間違ってたらスミマセンー

文字化けも以前似たような現象が出てたのですが、どうすりゃ直ったんだったかorz

yamanetoshi commented 10 years ago

文字化けの件、以下が参考になるのかどうか。。

https://github.com/RHGReadingOkinawa/rhgreadingokinawa.github.com/pull/9

Arakaki-Yuji commented 10 years ago

文字化けの件、参考になりました(^^*) ありがとうございます。

修正版コミットで、ローカル環境では問題なかったので他でも文字化けが起きないと良いのですが。。。

yamanetoshi commented 10 years ago

LGTM

yamanetoshi commented 10 years ago

merge してもいいかな。。

takumikinjo commented 10 years ago

:+1: