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

Fix syntax #1701

Merged
merged 8 commits into from
Oct 2, 2024
Merged

Fix syntax #1701

merged 8 commits into from
Oct 2, 2024

Conversation

hokorobi
Copy link
Contributor

No description provided.

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.

doc/ft_ada.jax について、ft_ada.txt のパッチ送付と、その取り込み後修正のほうが良いだろうと思われますが、それ以外はよさそうです

doc/builtin.jax Outdated Show resolved Hide resolved
doc/ft_ada.jax Outdated Show resolved Hide resolved
doc/ft_ada.jax Show resolved Hide resolved
doc/if_tcl.jax Outdated Show resolved Hide resolved
Copy link
Member

@h-east h-east left a comment

Choose a reason for hiding this comment

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

ft_ada.jax の変更は無しにして、代わりに if_ada.txt の差分を 本家helpのtypo報告スレ (Vim 9.1) にコピペしていただけるとありがたいです。

doc/builtin.jax Outdated
クが使うとエラーになってしまう場合にスキップするに便利である:
>
クが使うとエラーになってしまう場合にスキップするに便利である: >
Copy link
Member

Choose a reason for hiding this comment

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

LGTMなのですが、既存訳のtypoがあるのでこのPRで直してしまいましょう。
s/するに/するのに/

		クが使うとエラーになってしまう場合にスキップするのに便利である:
>

doc/if_tcl.jax Outdated Show resolved Hide resolved
@hokorobi
Copy link
Contributor Author

hokorobi commented Sep 29, 2024

ありがとうございます。
すみません、vim-jp/issues#1433 のコメントに気づかず ft_ada.txt の方は本体側にPRを送っちゃいました。

@h-east
Copy link
Member

h-east commented Sep 29, 2024

@hokorobi 正規の手順なので問題ないです👍。先ほど当該PRがmergeされたようなので、このPRに if_ada.txt の差分を含めて貰うと良いのかな?

@hokorobi
Copy link
Contributor Author

hokorobi commented Sep 29, 2024

ft_ada.txt に追加で修正した分と指摘いただいた分を反映してみました。

@tsuyoshicho
Copy link
Contributor

@hokorobi

ft_ada.txt に追加で修正した分と指摘いただいた分を反映してみました。

その修正分と明確にするため、ft_ada.txt の最新版もこのPRに含めてもらえますか?

そしたら、確認して approve しなおします

@tsuyoshicho tsuyoshicho self-requested a review September 29, 2024 13:26
doc/builtin.jax Outdated Show resolved Hide resolved
@hokorobi
Copy link
Contributor Author

よろしくお願いします。

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.

指摘のところを適用したらOKかと思います
全体としてはLGTM

en/ft_ada.txt Show resolved Hide resolved
doc/ft_ada.jax Outdated
@@ -48,13 +48,12 @@ Adaモードではユーザーが設定できるオプションがいくつか
トは|ft-ada-options|を参照。

それらを有効化するには、オプションに値を設定する。
あるオプションを有効化する例:
>
あるオプションを有効化する例: >
let ada_standard_types = 1
Copy link
Member

@h-east h-east Sep 30, 2024

Choose a reason for hiding this comment

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

インデントを原文と合わせてください。

doc/ft_ada.jax Outdated
let ada_standard_types = 1
>
<
無効化するには ":unlet" を使用する。例: >
unlet ada_standard_types = 1
Copy link
Member

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.

あ...見落した

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ありがとうございます。

@k-takata k-takata merged commit 08aa12d into vim-jp:master Oct 2, 2024
2 checks passed
@k-takata
Copy link
Member

k-takata commented Oct 2, 2024

thx

@hokorobi
Copy link
Contributor Author

hokorobi commented Oct 2, 2024

ご協力ありがとうございました。

@hokorobi hokorobi deleted the fix-syntax branch October 2, 2024 11:43
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