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 filetype.{txt,jax} #1138

Merged
merged 10 commits into from
Sep 29, 2022
Merged

Update filetype.{txt,jax} #1138

merged 10 commits into from
Sep 29, 2022

Conversation

ryicoh
Copy link
Contributor

@ryicoh ryicoh commented Sep 21, 2022

お願いします🙏

doc/filetype.jax Outdated Show resolved Hide resolved
doc/filetype.jax Outdated Show resolved Hide resolved
doc/filetype.jax Outdated Show resolved Hide resolved
doc/filetype.jax Outdated Show resolved Hide resolved
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.

前の指摘は、概要的に言ってしまったこともあり、子細な部分がうまくつたわっていませんでした。
訳がこなれてきているので、その訳に手を入れた状態を提示して指摘をしています。

あと、全体指摘としてWikiにある autofmtプラグイン でフォーマットしておくのは留意してください。

doc/filetype.jax Outdated Show resolved Hide resolved
doc/filetype.jax Outdated Show resolved Hide resolved
doc/filetype.jax Outdated Show resolved Hide resolved
@ryicoh
Copy link
Contributor Author

ryicoh commented Sep 23, 2022

あと、全体指摘としてWikiにある autofmtプラグイン でフォーマットしておくのは留意してください。

ci 欲しいですね〜

提案ありがとうございました!取り込みました!

ryicoh and others added 3 commits September 23, 2022 15:49
Co-authored-by: Tsuyoshi CHO <Tsuyoshi.CHO+github@Gmail.com>
Co-authored-by: Tsuyoshi CHO <Tsuyoshi.CHO+github@Gmail.com>
Co-authored-by: Tsuyoshi CHO <Tsuyoshi.CHO+github@Gmail.com>
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.

ひと通りレビューしました。

doc/filetype.jax Outdated Show resolved Hide resolved
doc/filetype.jax Outdated Show resolved Hide resolved
doc/filetype.jax Outdated Show resolved Hide resolved
For example, to set the dialect to a default of "fblite" but still allow for
any #lang directive overrides, use the following command: >

let g:freebasic_lang = "fblite"
Copy link
Member

Choose a reason for hiding this comment

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

インデントが既存のものと異なっています(タブ1つにすべき)。
原文の修正が必要ですので、専用スレッドにレスしました。
vim-jp/issues#1387 (comment)

ryicoh and others added 3 commits September 28, 2022 22:40
Co-authored-by: h_east <h.east.727@gmail.com>
Co-authored-by: h_east <h.east.727@gmail.com>
Co-authored-by: h_east <h.east.727@gmail.com>
@ryicoh
Copy link
Contributor Author

ryicoh commented Sep 28, 2022

@h-east
レビュー&提案ありがとうございました。!取り込ませていただきました 🙏

インデントが既存のものと異なっています(タブ1つにすべき)。

こちらは、原文に合わせてそのままの方がいいでしょうか?

@h-east
Copy link
Member

h-east commented Sep 28, 2022

@ryicoh

こちらは、原文に合わせてそのままの方がいいでしょうか?

はい。原文が修正されるまでは、そのままでOKです。

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.

細かい言葉遣いまわりで気になる箇所がなくはないんですが、それは全体的にいろいろあると思うので、今回これでいいと思います。

@tsuyoshicho tsuyoshicho merged commit ac15cd0 into vim-jp:master Sep 29, 2022
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