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.

Sign up for GitHub

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

Jump to bottom

Translate 3 tsconfig files to ja #978

Merged
merged 14 commits into from
Oct 21, 2020
Merged

Translate 3 tsconfig files to ja #978

merged 14 commits into from
Oct 21, 2020

Conversation

harapeko
Copy link
Contributor

@harapeko harapeko commented Aug 27, 2020 •
edited
Loading

@ghost
Copy link

ghost commented Aug 27, 2020 •
edited by ghost
Loading

CLA assistant check
All CLA requirements met.

@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.

@harapeko harapeko force-pushed the translate/ja-tsconfig-files branch from dc0ac63 to bf5c4b6 Compare September 2, 2020 20:13
@harapeko harapeko marked this pull request as ready for review September 2, 2020 20:38
@harapeko
Copy link
Contributor Author

harapeko commented Sep 2, 2020

レビューおねがいします！
@sasurau4, @Quramy, @Naturalclar, and @Takepepe

@Takepepe さんは @takefumi-yoshii さんにIDが変更されたのでしょうか？
そうであれば、.github/CODEOWNERS の方も更新しておいたほうが良さそうです

sasurau4
sasurau4 reviewed Sep 16, 2020
View reviewed changes
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/intro.md Outdated Show resolved Hide resolved
packages/tsconfig-reference/copy/ja/categories/Watch_Options_999.md Outdated Show resolved Hide resolved
packages/tsconfig-reference/copy/ja/categories/Watch_Options_999.md Outdated Show resolved Hide resolved
packages/tsconfig-reference/copy/ja/options/fallbackPolling.md Outdated
@@ -0,0 +1,11 @@
---
display: "フォールバック ポーリング"
oneline: "システムがネイティブのファイルウォッチャーを使い切ったら、ウォッチャー使うべきもの"
Copy link
Contributor

Choose a reason for hiding this comment

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

displayとonelineはあとでまとめてやろうということで全部残したままにしてあるので、一旦訳さなくても大丈夫です 👍

Copy link
Contributor Author

@harapeko harapeko Sep 30, 2020 •
edited
Loading

Choose a reason for hiding this comment

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

  • Watch_Options_999.md
  • fallbackPolling.md
    両方をもとに戻しました！
    f9fe3ce

packages/tsconfig-reference/copy/ja/options/fallbackPolling.md Outdated Show resolved Hide resolved
packages/tsconfig-reference/copy/ja/options/fallbackPolling.md Outdated Show resolved Hide resolved
packages/tsconfig-reference/copy/ja/options/fallbackPolling.md Outdated Show resolved Hide resolved
packages/tsconfig-reference/copy/ja/options/fallbackPolling.md Outdated
- `fixedPollingInterval`: 一定の間隔を開けて、1 秒間に数回、全ファイルをチェックします。
- `priorityPollingInterval`: 1 秒間に数回、全ファイルをチェックします。しかし、特定の種類のファイルにはヒューリスティックを使用して、他ファイルよりも低頻度でチェックします。
- `dynamicPriorityPolling`: 変更頻度の少ないファイルは、低頻度でチェックされるように動的キューを使用します。
- `synchronousWatchDirectory`: ディレクトリの遅延監視を無効にします。遅延監視は、沢山のファイル変更が一度に引き起こされる場合に有効です(例： `npm install` 実行による `node_modules` の変更)。しかし、このフラグは余り一般的でないセットアップの為にあるので、無効にしておくのが良いでしょう。
Copy link
Contributor

Choose a reason for hiding this comment

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

最後の文の意味が逆だと思われます。一般的にはonにしておいて必要に応じてoffにもできるという意味だと思われるので直訳してみました。直訳すぎてこなれてないのでもっと良い訳があると嬉しいです。

Suggested change
- `synchronousWatchDirectory`: ディレクトリの遅延監視を無効にします。遅延監視は、沢山のファイル変更が一度に引き起こされる場合に有効です(例： `npm install` 実行による `node_modules` の変更)。しかし、このフラグは余り一般的でないセットアップの為にあるので、無効にしておくのが良いでしょう。
- `synchronousWatchDirectory`: ディレクトリの遅延監視を無効にします。遅延監視は、沢山のファイル変更が一度に引き起こされる場合に有効です(例： `npm install` 実行による `node_modules` の変更)。しかし、あまり一般的でないセットアップの為に、このフラグで遅延監視機能を無効にしたくなることもあるかもしれません。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

次のように修正してみました！
5f4d050

harapeko and others added 8 commits October 1, 2020 00:32
@harapeko @sasurau4
Update packages/tsconfig-reference/copy/ja/intro.md
ccbdd49 
Co-authored-by: Daiki Ihara <sasurau4@gmail.com>
@harapeko @sasurau4
Update packages/tsconfig-reference/copy/ja/categories/Watch_Options_9…
00197a1 
…99.md

Co-authored-by: Daiki Ihara <sasurau4@gmail.com>
@harapeko @sasurau4
Update packages/tsconfig-reference/copy/ja/categories/Watch_Options_9…
501ddc9 
…99.md

Co-authored-by: Daiki Ihara <sasurau4@gmail.com>
@harapeko @sasurau4
Update packages/tsconfig-reference/copy/ja/options/fallbackPolling.md
d2c38be 
Co-authored-by: Daiki Ihara <sasurau4@gmail.com>
@harapeko @sasurau4
Update packages/tsconfig-reference/copy/ja/options/fallbackPolling.md
2123bc1 
Co-authored-by: Daiki Ihara <sasurau4@gmail.com>
@harapeko @sasurau4
Update packages/tsconfig-reference/copy/ja/options/fallbackPolling.md
67b7675 
Co-authored-by: Daiki Ihara <sasurau4@gmail.com>
@harapeko
restore display, online to en.
f9fe3ce
@harapeko
modify synchronousWatchDirectory to improved Japanese
5f4d050
@harapeko harapeko force-pushed the translate/ja-tsconfig-files branch from 9452b63 to 5f4d050 Compare September 30, 2020 16:16
@harapeko
Copy link
Contributor Author

harapeko commented Sep 30, 2020 •
edited
Loading

@sasurau4, @Quramy, @Naturalclar
再レビュー依頼になります。

作業内容

  • Suggestion取り込んだ
  • displayとonelineを元に戻した(後から一気に対応できるように)
  • synchronousWatchDirectory の訳を洗練しました

よろしくお願いします！

sasurau4
sasurau4 suggested changes Oct 5, 2020
View reviewed changes
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.

1点だけ 🙏

packages/tsconfig-reference/copy/ja/options/fallbackPolling.md Outdated Show resolved Hide resolved
@harapeko
Copy link
Contributor Author

harapeko commented Oct 9, 2020

@sasurau4, @Quramy, @Naturalclar
再レビュー依頼になります。

  • 頂いた1点のSuggestionを取り込みました
  • 同オプションの訳を改善しました(少し違和感を感じたため)

何度も日が空いてしまい、大変お手数おかけ致しますがよろしくお願いいたします。

sasurau4
sasurau4 approved these changes Oct 15, 2020
View reviewed changes
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.

めっちゃ長くなってしまいましたが、ありがとうございました！ 👍

LGTM

@harapeko
Copy link
Contributor Author

感謝です>w<ノ

@sasurau4
Copy link
Contributor

@orta Codeowners merging workflow failed because of unhandled promise rejection. Could you investigate it?
https://github.com/microsoft/TypeScript-Website/pull/978/checks?check_run_id=1257687413

@orta
Copy link
Contributor

orta commented Oct 21, 2020

Thanks @sasurau4 - I'll merge this for now and look 👍🏻

@orta orta merged commit c518439 into microsoft:v2 Oct 21, 2020
@orta orta mentioned this pull request Oct 21, 2020
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sasurau4 sasurau4 sasurau4 approved these changes

Assignees
No one assigned
Labels
translation
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@harapeko @sasurau4 @orta