-
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] translated contributing/announcements into ja #6530
base: main
Are you sure you want to change the base?
Conversation
6ad5a53
to
0a7f285
Compare
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.
@Msksgm I commented on two places. Thanks!
|
||
このリポジトリの内容にバグや問題を発見した場合、または改善をリクエストしたい場合は、[ドキュメントのイシューを作成][new-issue] してください。 | ||
|
||
セキュリティ上の問題を発見した場合は、イシューを作成する前に[セキュリティポリシー](https://github.com/open-telemetry/opentelemetry.io/security/policy) をお読みください。 |
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.
「お読みください」だけ唐突に丁寧さが上がってるので「確認してください」くらいのトーンで。原文にもそこまで丁寧なニュアンスはありません。
If you discover a security issue, read the Security Policy before opening an issue.
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.
daf3fe1
to
f6a84b0
Compare
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.
LGTM 🎉
I translated contributing/announcements into ja.
orginal https://opentelemetry.io/docs/contributing/announcements/
preview https://deploy-preview-6530--opentelemetry.netlify.app/ja/docs/contributing/announcements/