Skip to content

Update map.jax #387

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from Dec 10, 2018
Merged

Update map.jax #387

merged 2 commits into from Dec 10, 2018

Conversation

ghost
Copy link

@ghost ghost commented Dec 8, 2018

「スペース」を追加しました。

@h-east
Copy link
Member

h-east commented Dec 9, 2018

PRありがとうございます:+1:
1行は78カラム以内と決まっていますので、適宜改行を入れてください。
整形プラグインautofmtの導入をお願いします。

導入方法はwikiページのVim ドキュメント翻訳者の手引き を「autofmt」で検索したり、
#54 (comment) を参照ください。

改行しました。
@ghost
Copy link
Author

ghost commented Dec 9, 2018

改行を入れました。
(ブラウザで直接編集しています。)

@h-east
Copy link
Member

h-east commented Dec 9, 2018

(ghostになっちゃった...)
そうか、ブラウザで日本語ヘルプ見ていて間違いに気付いてその場で修正してPRするのは「仕組みとしては」サクッとできるようになったけど、改行が絡んでくるととたんに敷居が上がってしまうんですね。

All>
これ、どうしたらいいんでしょうね?
指摘が正しければmergeして、改行絡みの整形はこちらですればいいのかな。
でも、1度くらいは「改行入れてください」ってお願いしたい気もしますね。
とりあえず、PR数が少ないうちはこちらで整形しますか。

@h-east h-east merged commit 9417a01 into vim-jp:master Dec 10, 2018
@h-east
Copy link
Member

h-east commented Dec 10, 2018

改行位置を変更してmergeしました。

@mnishz
Copy link
Member

mnishz commented Dec 11, 2018

oh... 何故。。。
確かに前にブラウザからお試しでやったときは、結局一度 Vim を起動して改行チェックしましたね。
今くらいのペースだと、こっちで改行を修正する手間よりも、気軽に投稿してもらえる恩恵のほうが大きいような気がします。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants