-
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] update content/ja/_index #6351
base: main
Are you sure you want to change the base?
Conversation
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 the update! I left a nit fix request.
content/ja/_index.md
Outdated
OpenTelemetryは、[Spring](https://spring.io)、[ASP.NET Core](https://docs.microsoft.com/aspnet/core)、[Express](https://expressjs.com)、[Quarkus](https://quarkus.io)などの一般的なライブラリやフレームワークとインテグレートされています! | ||
インストールとインテグレーションは、数行のコードで行えるほど簡単です。 | ||
OpenTelemetry は多くの人気のライブラリーとフレームワークに[インテグレート][instrumentation]されています。 | ||
そして、コードベースとゼロコードの[計装][integrates]をサポートしています。 |
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.
「コードベース」だと意味が変わってしまうので、ここでの文脈で意訳して「手動」で良いと思います。
https://ja.wikipedia.org/wiki/%E3%82%B3%E3%83%BC%E3%83%89%E3%83%99%E3%83%BC%E3%82%B9
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 your review.
I fixed it in a bellow commit.
eaf1f59
to
338691f
Compare
338691f
to
86cf870
Compare
I updated content/ja/_index page.
ref: #6172.