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 indent.{txt,jax} #1448

Merged
merged 2 commits into from
Mar 10, 2024
Merged

Update indent.{txt,jax} #1448

merged 2 commits into from
Mar 10, 2024

Conversation

mityu
Copy link
Member

@mityu mityu commented Mar 10, 2024

翻訳しました。よろしくお願いします。

@mityu
Copy link
Member Author

mityu commented Mar 10, 2024

vim-jp/issues#1387 (comment)

ここで報告したコードブロックの閉じ忘れの話なんですが、内容に関わるものではなく見た目に関わるだけのもので、この修正がなくても大した問題にはならないだろうと思ったので、本家の方にこの修正が入ったらその時に jax の方も修正すれば十分だろうということで jax の方にこの修正を独自に取り込むようなことはしていません。なので、もし「いややっぱりこの修正は jax の方にはこのタイミングで取り込んでおいてほしい」ということがあれば一言お願いします。

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

1点確認をおねがいします。

doc/indent.jax Outdated
YAML *ft-yaml-indent*

デフォルトでは、yaml のインデントスクリプトは複数行のスカラーの検出を行おうと
しない。もしこれを有効化したい場合は、以下の変数をセットすること: >
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
Member Author

Choose a reason for hiding this comment

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

確かに設定となってますね。修正しました。

@tsuyoshicho tsuyoshicho requested a review from h-east March 10, 2024 07:48
@h-east h-east merged commit 1787bd9 into vim-jp:master Mar 10, 2024
2 checks passed
@h-east
Copy link
Member

h-east commented Mar 10, 2024

Thanks 👍

@mityu mityu deleted the update-indent branch March 10, 2024 15:55
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

3 participants