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

Update/New userfunc.{txt,jax} #1155

Merged
merged 11 commits into from
Nov 11, 2022

Conversation

tsuyoshicho
Copy link
Contributor

@tsuyoshicho tsuyoshicho commented Oct 9, 2022

翻訳(+evalからの切り出し)をしました、よろしくおねがいします。

註: evalのPR後 rebase するまでは helptag がコンフリクトする
require #1153

typo(と思うもの) vim-jp/issues#1387 (comment)

@tsuyoshicho tsuyoshicho self-assigned this Oct 9, 2022
@tsuyoshicho tsuyoshicho requested review from h-east and k-takata and removed request for h-east and k-takata October 9, 2022 08:48
@tsuyoshicho tsuyoshicho force-pushed the update/20221009/userfunc branch 3 times, most recently from 5b0be82 to 4ec5669 Compare October 10, 2022 03:24
@tsuyoshicho tsuyoshicho marked this pull request as ready for review October 10, 2022 03:25
@k-takata
Copy link
Member

切り出した部分と、今回新規に翻訳した部分が分かれていればレビューしやすかったのですが。

@tsuyoshicho
Copy link
Contributor Author

あー、コミット的に混ってしまっていますね...すみません。
最初の持ち込み時点で、かなり入り組んだ差があったので、気付いたら混ぜてしまっていましたね...

doc/userfunc.jax Outdated Show resolved Hide resolved
doc/userfunc.jax Outdated Show resolved Hide resolved
doc/userfunc.jax Outdated Show resolved Hide resolved
doc/userfunc.jax Outdated Show resolved Hide resolved
doc/userfunc.jax Outdated Show resolved Hide resolved
doc/userfunc.jax Outdated Show resolved Hide resolved
doc/userfunc.jax Outdated
Comment on lines 417 to 418
In |Vim9| script there is also an autoload mechanism for imported scripts, see
|import-autoload|.
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 Author

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.

対応しました。

doc/userfunc.jax Outdated Show resolved Hide resolved
doc/userfunc.jax Outdated
:let foo#bar#toggle = 1
:call foo#bar#func()

オートロードスクリプト内で定義されるはずの関数を呼んだがスクリプト内で関数が定
Copy link
Member

Choose a reason for hiding this comment

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

when you make a mistake の訳が抜けています。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

対応しました。

doc/userfunc.jax Outdated Show resolved Hide resolved
doc/userfunc.jax Outdated Show resolved Hide resolved
doc/userfunc.jax Outdated
定義されるようにすることができる。これには2つの方法がある: 自動コマンドによ
る方法と、'runtimepath' 内の "autoload" ディレクトリによる方法である。

In |Vim9| script there is also an autoload mechanism for imported scripts, see
|import-autoload|.
Vim9| script ではインポートすることによるスクリプトの自動読み込みメカニズムも
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
Vim9| script ではインポートすることによるスクリプトの自動読み込みメカニズムも
|Vim9| script ではインポートすることによるスクリプトの自動読み込みメカニズムも

| が抜けています。
"imported scripts" はそのまま「インポートされたスクリプト」でよい気もしますが。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ぶ、たしかにそうなので直します。
(見直したときはあった気がしたのに)

doc/userfunc.jax Outdated Show resolved Hide resolved
@tsuyoshicho
Copy link
Contributor Author

前置詞 -> プリフィックスの対応もします。

@tsuyoshicho
Copy link
Contributor Author

@k-takata san

11月はほとんど時間取れないと思います。私のApprove待ちだけ状態のPRはmergeしていただいて構いません。

とのことなので、追加部分を確認いただいて、問題なければマージという方向でどうでしょうか?

@k-takata
Copy link
Member

#1153 をマージしたのでリベースしていただけますか。

@tsuyoshicho
Copy link
Contributor Author

@k-takata san
リベースしました。

@k-takata k-takata merged commit 5cff983 into vim-jp:master Nov 11, 2022
@k-takata
Copy link
Member

thx

@tsuyoshicho tsuyoshicho deleted the update/20221009/userfunc branch November 11, 2022 13:09
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