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] translate contributing/issues into ja #6186

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

Conversation

Msksgm
Copy link
Contributor

@Msksgm Msksgm commented Feb 3, 2025

translated contributing/issues page.

@opentelemetrybot opentelemetrybot requested review from a team February 3, 2025 12:38
@Msksgm Msksgm force-pushed the ja/contributing/issues branch 11 times, most recently from ca1854a to b32129d Compare February 5, 2025 11:38
@Msksgm Msksgm marked this pull request as ready for review February 5, 2025 11:39
@Msksgm Msksgm requested a review from a team as a code owner February 5, 2025 11:39
@Msksgm Msksgm force-pushed the ja/contributing/issues branch 2 times, most recently from a5b14a5 to ab81a6c Compare February 8, 2025 00:41
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 a series of your contribution, @Msksgm! Though I asked a couple of tiny changes, it looks great!

### 優れたイシューの作成方法 {#how-to-file-great-issues}

イシューを作成する際には、以下の点に注意してください。
Keep the following in mind when filing an issue:
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.

Sorry and thank you for your mistake.
I fixed a bellow commit.

bbc24aa


- 明確な説明を提供してください。何が欠けていて、古くなっていて、間違っていて改善を必要としているのか説明してください。
- ユーザーへの具体的な影響を説明してください。
- イシューの範囲は合理的な範囲に制限してください。問題の範囲が大きい場合は、小さなイシューに分割してください。例えば、「セキュリティドキュメントを修正する」は広すぎますが、「『ネットワークアクセスの制限』のトピックに詳細を追加する」は具体的で実行しやすいです。
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.

@ymotongpoo

Thank you for pointing out.
I fixed a bellow commit.

8059b20

- ユーザーへの具体的な影響を説明してください。
- イシューの範囲は合理的な範囲に制限してください。問題の範囲が大きい場合は、小さなイシューに分割してください。例えば、「セキュリティドキュメントを修正する」は広すぎますが、「『ネットワークアクセスの制限』のトピックに詳細を追加する」は具体的で実行しやすいです。
- 新しいイシューに関連していて似たようなイシューが存在していないか探してください。
- 新しいイシューがほかのイシューやプルリクエストに関連している場合は、該当する URL 全文を記載するか、`#` をつけてイシュー番号やプルリクエスト番号を記述してください。例えば、`Introduced by #987654` です。
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.

@ymotongpoo

Thank you for pointing out.
I fixed a bellow commit.

8059b20

@Msksgm Msksgm force-pushed the ja/contributing/issues branch from ab81a6c to 8059b20 Compare February 10, 2025 07:05
@opentelemetrybot opentelemetrybot requested a review from a team February 10, 2025 07:05
@Msksgm
Copy link
Contributor Author

Msksgm commented Feb 10, 2025

@ymotongpoo

Thank you for your review.
I followed your advices.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Development

Successfully merging this pull request may close these issues.

2 participants