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

[ja]: translation of demo/tests and demo/forking pages into ja #5981

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

yugo-ibuki
Copy link
Contributor

@yugo-ibuki yugo-ibuki commented Jan 19, 2025

I translated demo/tests and demo/forking pages.

@yugo-ibuki yugo-ibuki requested a review from a team as a code owner January 19, 2025 02:13
@yugo-ibuki yugo-ibuki changed the title [ja]: translation of tests and forking pages [ja]: translation of demo/tests and demo/forking pages Jan 19, 2025
@opentelemetrybot opentelemetrybot requested review from a team January 19, 2025 02:13
default_lang_commit: 99b15da7137cde4a6352576768f4c9203170d569
---

[demo repository][]は、OpenTelemetryを使用した実装例を紹介するためのツールとして、フォークして使用できるように設計されています。
Copy link
Contributor Author

@yugo-ibuki yugo-ibuki Jan 19, 2025

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.

https://github.com/open-telemetry/opentelemetry.io/blob/main/content/en/docs/demo/forking.md?plain=1#L6

Copy link
Contributor

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?

Copy link
Contributor Author

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!

#6004

@yugo-ibuki yugo-ibuki changed the title [ja]: translation of demo/tests and demo/forking pages [ja]: translation of demo/tests and demo/forking pages into ja Jan 19, 2025
Copy link
Contributor

@ymotongpoo ymotongpoo left a 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を使用した実装例を紹介するためのツールとして、フォークして使用できるように設計されています。
Copy link
Contributor

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?

## フォークメンテナーへの提案 {#suggestions-for-fork-maintainers}

- デモによって発信または収集されるテレメトリデータを強化したい場合は、変更内容をこのリポジトリにバックポートすることを強く推奨します。
ベンダーや実装固有の変更については、コードの根本的な変更よりも、設定によってパイプライン内でテレメトリを変更する戦略が望ましいです。
Copy link
Contributor

Choose a reason for hiding this comment

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

ここでは strategy は「方針」のほうが良いかもしれません

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done at c3162ca


## フォークメンテナーへの提案 {#suggestions-for-fork-maintainers}

- デモによって発信または収集されるテレメトリデータを強化したい場合は、変更内容をこのリポジトリにバックポートすることを強く推奨します。
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
Contributor Author

Choose a reason for hiding this comment

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

Done at d8f5106

- デモによって発信または収集されるテレメトリデータを強化したい場合は、変更内容をこのリポジトリにバックポートすることを強く推奨します。
ベンダーや実装固有の変更については、コードの根本的な変更よりも、設定によってパイプライン内でテレメトリを変更する戦略が望ましいです。
- 置き換えるのではなく、拡張してください。
既存のAPIと連携する全く新しいサービスを追加することは、テレメトリの変更では実現できないベンダー固有またはツール固有の機能を追加する優れた方法です。
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
Contributor Author

Choose a reason for hiding this comment

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

Done at d8f5106

@opentelemetrybot opentelemetrybot requested a review from a team January 21, 2025 13:27
@yugo-ibuki yugo-ibuki requested a review from ymotongpoo January 21, 2025 13:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

2 participants