Skip to content

[ja translation] tsconfig-reference 9 files #1432

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

Merged
merged 6 commits into from
Jan 25, 2021

Conversation

@github-actions
Copy link
Contributor

Thanks for the PR!

This section of the codebase is owned by @sasurau4, @Quramy, @Naturalclar, and @Takepepe - if they write a comment saying "LGTM" then it will be merged.

Copy link
Contributor Author

@uraway uraway left a comment

Choose a reason for hiding this comment

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

bundledPackageName.mdが原因でCIが失敗していた & TODOリストにも載っていなかったので削除

// ^?

// 宣言されていませんが、インデックス
// シグネチャのおかげで、stringとして扱われます
Copy link
Contributor

Choose a reason for hiding this comment

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

これも元のドキュメントが微妙に間違っているので、PRを出してもらえると 👍
上の文書とまるっきり一緒なので、noUncheckedIndexedAccessの使い所が分からなくなってしまいました :)

Suggested change
// シグネチャのおかげで、stringとして扱われます
// シグネチャのおかげで、string | undefinedとして扱われます

Copy link
Contributor Author

Choose a reason for hiding this comment

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

僕も最初は混乱したのですが、こちらは前述のコードブロックの抜粋だと思うので、コメントが全く同じなのは意図したものではないでしょうか

twoslashでは↓のように表示されるので、ユーザーが読む分には大丈夫かなと思ったのですが、どうでしょうか
image

Copy link
Contributor

Choose a reason for hiding this comment

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

Discordで聞いてみたので、それの返信を待ちましょう

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

@sasurau4 sasurau4 left a comment

Choose a reason for hiding this comment

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

遅くなりましたが見ました!
packages/tsconfig-reference/copy/ja/sections/watchOptions.md がempty fileですが、ミスでしょうか?

- `fixedPollingInterval`: すべてのファイルの変更を一定間隔で毎秒数回チェックします。
- `priorityPollingInterval`: すべてのファイルの変更を毎秒数回チェックしますが、ヒューリスティックスを使用して他のファイルよりも少ない頻度で特定のタイプのファイルをチェックします。
- `dynamicPriorityPolling`: 変更頻度の低いファイルがチェックされる頻度が低くなるような動的なキューを使用します。
- `useFsEvents` (デフォルト): オペレーティングシステム/ファイルシステムのネイティブイベントの使用をファイルの変更に試みます。
Copy link
Contributor

Choose a reason for hiding this comment

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

上と同様

Co-authored-by: Daiki Ihara <sasurau4@gmail.com>
@uraway
Copy link
Contributor Author

uraway commented Dec 24, 2020

レビューありがとうございます。watchOptions.mdは、オリジナルのファイルが空だったので気づかずにそのままコミットしてました。Discordで聞いてみます

uraway and others added 2 commits December 27, 2020 11:01
Co-authored-by: Daiki Ihara <sasurau4@gmail.com>
@uraway
Copy link
Contributor Author

uraway commented Dec 27, 2020

watchOptions.mdは空のままで良いとのことです

@orta
Copy link
Contributor

orta commented Jan 25, 2021

I think this PR just needs a final review?

Otherwise I'll close this PR at the end of the week, because we're moving these PRs into their own repos re: #1480

@uraway
Copy link
Contributor Author

uraway commented Jan 25, 2021

I am not the owner of the codebase, but I think this PR has passed all reviews

@orta
Copy link
Contributor

orta commented Jan 25, 2021

👍🏻

@orta orta merged commit 147fa63 into microsoft:v2 Jan 25, 2021
@uraway uraway deleted the feature/translation-tsconfig-files branch January 25, 2021 18:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants