-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
[ja]: translation of demo/tests and demo/forking pages into ja #5981
base: main
Are you sure you want to change the base?
[ja]: translation of demo/tests and demo/forking pages into ja #5981
Conversation
default_lang_commit: 99b15da7137cde4a6352576768f4c9203170d569 | ||
--- | ||
|
||
[demo repository][]は、OpenTelemetryを使用した実装例を紹介するためのツールとして、フォークして使用できるように設計されています。 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I found there is no link url here.
So I follow this sentence as it is.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
no worries. would you mind creating an issue that addresses this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I created the issue!
If you think you need more info, let me know!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for another contribution, @yugo-ibuki! I added minor requests for approval.
default_lang_commit: 99b15da7137cde4a6352576768f4c9203170d569 | ||
--- | ||
|
||
[demo repository][]は、OpenTelemetryを使用した実装例を紹介するためのツールとして、フォークして使用できるように設計されています。 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
no worries. would you mind creating an issue that addresses this?
content/ja/docs/demo/forking.md
Outdated
## フォークメンテナーへの提案 {#suggestions-for-fork-maintainers} | ||
|
||
- デモによって発信または収集されるテレメトリデータを強化したい場合は、変更内容をこのリポジトリにバックポートすることを強く推奨します。 | ||
ベンダーや実装固有の変更については、コードの根本的な変更よりも、設定によってパイプライン内でテレメトリを変更する戦略が望ましいです。 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ここでは strategy は「方針」のほうが良いかもしれません
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done at c3162ca
content/ja/docs/demo/forking.md
Outdated
|
||
## フォークメンテナーへの提案 {#suggestions-for-fork-maintainers} | ||
|
||
- デモによって発信または収集されるテレメトリデータを強化したい場合は、変更内容をこのリポジトリにバックポートすることを強く推奨します。 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
「テレメトリー」(長音あり)に統一しています
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done at d8f5106
content/ja/docs/demo/forking.md
Outdated
- デモによって発信または収集されるテレメトリデータを強化したい場合は、変更内容をこのリポジトリにバックポートすることを強く推奨します。 | ||
ベンダーや実装固有の変更については、コードの根本的な変更よりも、設定によってパイプライン内でテレメトリを変更する戦略が望ましいです。 | ||
- 置き換えるのではなく、拡張してください。 | ||
既存のAPIと連携する全く新しいサービスを追加することは、テレメトリの変更では実現できないベンダー固有またはツール固有の機能を追加する優れた方法です。 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
同上
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done at d8f5106
I translated demo/tests and demo/forking pages.