16 arakaki yuji#29
Merged
Merged
Conversation
added 2 commits
May 24, 2014 13:57
Contributor
|
👍 |
Contributor
|
手元で動作確認してみたのですが
について対処した方が良くなると思いますが、如何でしょ。そのまま反映、でも良いと思っています。 |
Contributor
|
自環境では文字化けは見れませんでした。URLは、ハイパーリンクできたほうがいいかもですね確かに。この機にリンクだけ直されてはいかがでしょう。面白いかもしれません、プルリクエストの出し直しなるものをみんなでどう扱うか等。この場合、単にブランチで更新して再度プルリクエストでいいのですかね？やったことないためwktk |
Author
|
レビューありがとうございます。 プルリクエストの出し直しは、kinjoさんの提案して頂いた通りブランチで更新して再度プルリクエストで良いかな？と思っております。 |
Contributor
|
修正版はそのまま push すればこの pull request に反映されるはずですよー 16-Arakaki-Yuji branch に修正をもりこんで再度 push という意味です確か再度作り直さなくても良かったはずですので試してみて下さい。 間違ってたらスミマセンー文字化けも以前似たような現象が出てたのですが、どうすりゃ直ったんだったかorz |
Contributor
|
文字化けの件、以下が参考になるのかどうか。。 |
・文字化けの修正 その他、以下も修正しました。 ・URLをハイパーリンクに ・コマンド引用部分の記述 ・Chefで使われる言葉が読みづらいので改行追加
Author
|
文字化けの件、参考になりました(^^*) 修正版コミットで、ローカル環境では問題なかったので他でも文字化けが起きないと良いのですが。。。 |
Contributor
|
|
Contributor
|
merge してもいいかな。。 |
Contributor
|
👍 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
もくもく会から数日過ぎましたが、もくもくした内容をまとめましたのでプルリクを送ります(^^*)