Skip to content

Conversation

Copy link
Member

@Msksgm Msksgm left a comment

Choose a reason for hiding this comment

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

[nits]

何点かコメントしました。

@opentelemetrybot opentelemetrybot requested a review from a team September 7, 2025 10:22
@kohbis kohbis requested a review from Msksgm September 7, 2025 12:57
@kohbis
Copy link
Contributor Author

kohbis commented Sep 7, 2025

@Msksgm ありがとうございます 🙇 修正しました

Copy link
Member

@Msksgm Msksgm left a comment

Choose a reason for hiding this comment

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

LGTM

@Msksgm Msksgm added the missing:docs-approval Co-owning SIG has provided approval, PR needs approval from docs maintainer label Sep 7, 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.

nit change

@opentelemetrybot opentelemetrybot requested a review from a team September 9, 2025 12:19
@kohbis
Copy link
Contributor Author

kohbis commented Sep 9, 2025

@ymotongpoo ありがとうございます 🙇 取り込みました 🙏

@svrnm svrnm enabled auto-merge September 10, 2025 08:22
@svrnm
Copy link
Member

svrnm commented Sep 10, 2025

@ymotongpoo PTAL if your suggestions are addressed, thank you!

@vitorvasc vitorvasc added missing:sig-approval Co-owning SIG didn't provide an approval and removed missing:docs-approval Co-owning SIG has provided approval, PR needs approval from docs maintainer labels Sep 10, 2025
@otelbot-docs otelbot-docs bot requested review from a team November 22, 2025 08:10
@svrnm svrnm added this pull request to the merge queue Nov 22, 2025
Merged via the queue into open-telemetry:main with commit 9c45ace Nov 22, 2025
22 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lang:ja missing:sig-approval Co-owning SIG didn't provide an approval

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

5 participants