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

[ja] translate contributing/blog into ja #6521

Merged
merged 1 commit into from
Mar 12, 2025
Merged

[ja] translate contributing/blog into ja #6521

merged 1 commit into from
Mar 12, 2025

Conversation

Msksgm
Copy link
Contributor

@Msksgm Msksgm commented Mar 11, 2025 •
edited
Loading

@opentelemetrybot opentelemetrybot requested review from a team March 11, 2025 13:16
@Msksgm Msksgm force-pushed the ja/contributing/blog branch from 5308c6f to 0e0b9a9 Compare March 11, 2025 13:17
@Msksgm Msksgm marked this pull request as ready for review March 11, 2025 13:31
@Msksgm Msksgm requested a review from a team as a code owner March 11, 2025 13:31
@Msksgm Msksgm force-pushed the ja/contributing/blog branch 3 times, most recently from 4f7766b to cfdbde2 Compare March 11, 2025 13:48
ymotongpoo
ymotongpoo requested changes Mar 11, 2025
View reviewed changes
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 your contribution. I request a couple of changes.

@Msksgm Msksgm force-pushed the ja/contributing/blog branch from cfdbde2 to 4e4e60f Compare March 11, 2025 21:23
@opentelemetrybot opentelemetrybot requested a review from a team March 11, 2025 21:24
ymotongpoo
ymotongpoo approved these changes Mar 12, 2025
View reviewed changes
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.

LGTM

@ymotongpoo ymotongpoo added the docs-approval-missing Co-owning SIG has provided approval, PR needs approval from docs maintainer label Mar 12, 2025
@Msksgm Msksgm force-pushed the ja/contributing/blog branch from 4e4e60f to a75fd7b Compare March 12, 2025 01:26
@opentelemetrybot opentelemetrybot requested a review from a team March 12, 2025 01:26
@chalin chalin added this pull request to the merge queue Mar 12, 2025
Merged via the queue into open-telemetry:main with commit 288f6e3 Mar 12, 2025
18 checks passed
cijothomas pushed a commit to cijothomas/opentelemetry.io that referenced this pull request Mar 14, 2025
@Msksgm @cijothomas
[ja] translate contributing/blog into ja (open-telemetry#6521)
1f5ec0f
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ymotongpoo ymotongpoo ymotongpoo approved these changes

@chalin chalin chalin approved these changes

Assignees
No one assigned
Labels
docs-approval-missing Co-owning SIG has provided approval, PR needs approval from docs maintainer lang:ja
Projects
SIG Comms: Docs Translation
Status: Done
SIG Comms: PRs & Issues
Status: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Msksgm @ymotongpoo @chalin