Skip to content

Update ft_rust.{txt,jax} #1600

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 4 commits into from
Jul 6, 2024
Merged

Update ft_rust.{txt,jax} #1600

merged 4 commits into from
Jul 6, 2024

Conversation

h-east
Copy link
Member

@h-east h-east commented Jun 30, 2024

とりあえず、現状の ft_rust.txt を翻訳して ft_rust.jax を更新しました。

@h-east
Copy link
Member Author

h-east commented Jun 30, 2024

vim_devでおこなった runtime/doc/ft_rust.txt の修正がなかったことになった件の流れと現状

  1. Vimに同梱されているRust関連のプラグインは https://github.com/rust-lang/rust.vim で開発されている。
  2. 上記ファイルタイププラグインのドキュメント doc/rust.txt がリネームされて、https://github.com/vim/vimruntime/doc/ft_rust.txt に同梱されている。
  3. そんなことは runtime/doc/ft_rust.txt を見ても分からない。
  4. 以前、runtime/doc/ft_rust.txt のtypoや微修正を何度かvim_devに送った。
  5. Bram氏もこのファイルの管理元が rust-lang/rust.vim だと言うのを忘れていて(?)、そのまま修正を取り込んだ。(rust-lang/rust.vim には未連絡)
  6. ある日、rust-lang/rust.vim の人がrust.vim関連のファイルの更新PRを送ってきた。
  7. そのままmergeされて、vim_dev でおこなった vim/runtime/doc/ft_rust.txt の修正分がなかったことになった。
  8. ここでやっと、「1.」の存在に気づいたので、連絡 した。
  9. 回答 があって、(意訳)「知らんがな、こっちに責任はないよ、君が rust-lang/rust.vim にPRしなよ」と言われる。
  10. 心折れたので放置。runtime/doc/ft_rust.jax も放置。
  11. (8か月経過...)
  12. とりあえず、現状の runtime/doc/ft_rust.txt を翻訳して ft_rust.jax を更新。 ←今ココ

@mityu
Copy link
Member

mityu commented Jul 5, 2024

おつかれさまです。一通り見て気になった点などをコメントしました。

@h-east h-east requested a review from mityu July 5, 2024 17:10
Copy link
Contributor

@tsuyoshicho tsuyoshicho left a comment

Choose a reason for hiding this comment

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

深く見れなかったのですが、大丈夫そう
LGTM

@h-east h-east merged commit d034ec3 into master Jul 6, 2024
3 checks passed
@h-east h-east deleted the hh-update-ft_rust branch July 6, 2024 05:47
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.

3 participants