Skip to content
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

Just update the Vim version in the first line from 8.0 to 8.1 #210

Merged
merged 2 commits into from
Jun 2, 2018

Conversation

mnishz
Copy link
Member

@mnishz mnishz commented Jun 1, 2018

Issue #207 に記載されているファイルのうち、単に先頭行のVim versionを8.0から8.1に上げればよいものだけをピックアップして修正しました。
一部文字コードの扱いに自信がないものがあったので、以下のファイルは省きました。

  • digraph.jax
  • farsi.jax
  • pi_zip.jax
  • hebrew.jax (そもそもファイルが存在しない?)

以下PRとは直接関係ないコメント。
Twitterの翻訳者募集を見て、貢献したいと思い挑戦しました。
Vimの翻訳もGitHubも不慣れなので、まずは簡単なものだけに留めました。
WikiのWorkflowGuideには一通り目を通したつもりですが、
もし手順やマナーに関して不適切なものがあればご教示いただけると幸いです。

doc/uganda.jax Outdated
@@ -1,4 +1,4 @@
*uganda.txt* For Vim バージョン 8.0. Last change: 2013 Jul 06
*uganda.txt* For Vim バージョン 8.1. Last change: 2013 Jul 06
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

en/uganda.txt では日付も変更されています。
2018 May 17

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

あれ、本当ですね。失礼いたしました。。
修正しました。ご指摘どうもありがとうございました。

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

いいと思います。

@h-east
Copy link
Member

h-east commented Jun 2, 2018

@mnishz PRありがとうございます。
内容問題ないと思います:+1:

@uakms
Copy link
Contributor

uakms commented Jun 2, 2018

farsi.txt 以外は UTF-8 になっていると思います。

nkf --guess pi_zip.txt
EUC-JP (LF)

となりますが……
(エディタで開くと UTF-8 判定。よくわかりません)

@h-east h-east merged commit 702e858 into vim-jp:master Jun 2, 2018
@h-east
Copy link
Member

h-east commented Jun 2, 2018

@mnishz Thanks!

@mnishz mnishz deleted the update_vim_version branch June 3, 2018 06:32
@mnishz
Copy link
Member Author

mnishz commented Jun 3, 2018

@h-east
Mergeどうもありがとうございました。
#207 の表も更新したほうがいいのかな?と思いましたが、権限がなさそうですね。
引き続き合間を見て簡単そうなものから翻訳進めてみます。

@h-east
Copy link
Member

h-east commented Jun 3, 2018

@mnishz

#207 の表も更新したほうがいいのかな?と思いましたが、権限がなさそうですね。
引き続き合間を見て簡単そうなものから翻訳進めてみます。

(もうお気付きと思いますが)表は今日のお昼すぎに更新済です。
権限 → vim-jpのwriterチームに招待しました:+1:

@mnishz
Copy link
Member Author

mnishz commented Jun 4, 2018

@h-east
表の更新と権限付与どうもありがとうございました。
更新できるか挑戦してみましたが、編集ボタンが見つかりませんでした。
権限の問題と考えていたのですが、そもそも他人が書き始めたコメントは
編集できない???のかもしれませんね。

@h-east
Copy link
Member

h-east commented Jun 4, 2018

更新できるか挑戦してみましたが、編集ボタンが見つかりませんでした。

権限関係、私もOwnerなんですが、「誰にどこまで」の基準が分からないので他のおっさんにお任せします。
表は私の方で更新しますよ:+1:

@mattn
Copy link
Member

mattn commented Jun 5, 2018

おっさんです。もし権限足らなかったら教えて下さい。

@mnishz
Copy link
Member Author

mnishz commented Jun 5, 2018

@mattn さん、
取り立てて困っているというわけではないので今のところ大丈夫です。
ご配慮どうもありがとうございます。

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.

4 participants