Skip to content

Update builtin.{txt,jax} - #2824

Merged
h-east merged 5 commits into
vim-jp:masterfrom
h-east:update-builtin2
Oct 17, 2026
Merged

h-east merged 5 commits into
vim-jp:masterfrom
h-east:update-builtin2

Conversation

@h-east

@h-east h-east commented Oct 9, 2026 •

Copy link
Copy Markdown
Member

getinfo() は原文が変過ぎたので、全面的に見直しして修正しました。PR済み。
で、日本語訳は修正後の原文をもとにして作業しましたが、en/builtin.txt は当然古いままです。
ですので、getinfo() のレビューはしないようお願いします。

(ADD:)
PR mergeされたので getinfo() のレビューしていただいておkです。

@tsuyoshicho tsuyoshicho left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

気付ける範囲で

Comment thread doc/builtin.jax Outdated

{opts} に |vim9-types| のリストを持つ "argtypes" 項目
があるとき、"returns" はそれらの型の引数で呼んだときの
型になる。List に無い必須の引数は "any" とみなす。

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

リンクしてる (|List|) であれば英語ですが、素だと全体はリストですね

全部英語でいく方向ならOKですが、そうでないなら微調整?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

fixed.

Comment thread doc/builtin.jax Outdated
{name} は "v:count" のように "v:" で始まる必要
がある。
name "v:" を含む変数名
type the type as a String, see |vim9-types|

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

次の行で翻訳してるようなので、削除漏れかな

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

ですね。Thanks.

@h-east

h-east commented Oct 17, 2026

Copy link
Copy Markdown
Member Author

fixed.
それと、PR mergeされたので getinfo() のレビューしていただいておkです。

@tsuyoshicho tsuyoshicho left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

他はOKとおもいます
LGTM

Comment thread doc/builtin.jax Resolved
Co-authored-by: Tsuyoshi CHO <Tsuyoshi.CHO+github@Gmail.com>
@h-east
h-east merged commit de44f15 into vim-jp:master Oct 17, 2026
4 checks passed
@h-east
h-east deleted the update-builtin2 branch October 17, 2026 10:19
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.

2 participants