Skip to content

Conversation

@f-yoshihara
Copy link
Contributor

@f-yoshihara f-yoshihara commented Nov 18, 2025

やったこと

GitHubのPRテンプレート (.github/pull_request_template.md) を新規追加しました。

このテンプレートにより、PRを作成する際に以下の項目が自動的に表示されるようになります:

  • やったこと: 変更内容の説明
  • やっていないこと: あえてやっていない項目とその理由
  • 気になっていること: 不安な点や相談したい点
  • その他: 関連PRや補足情報
  • 動作確認: 実施状況、前準備、確認手順

期待される効果

  • PRの品質向上とレビューの効率化
  • PRのdescriptionに必要な要素をトレイニーに意識してもらいやすくする

やっていないこと

なし

気になっていること

コメントとして記載しました

その他

なし

動作確認

PRテンプレートの仕組み自体はGitHubの機能なので、動作確認は割愛

実施状況

  • ローカル開発環境での動作確認を実施済み

前準備

なし

確認手順

なし

f-yoshihara and others added 2 commits November 17, 2025 17:10
変更内容、実施内容、気になっていること、動作確認手順などの項目を含めたテンプレートを追加し、PRの品質向上とレビューの効率化を目指します。

🤖 Generated with Claude Code

Co-Authored-By: Claude <[email protected]>
「やっていないこと(後でやりたいこと)」を「やっていないこと」に変更し、あえてやっていないことがあればその理由を記載するように促すメッセージに更新しました。

🤖 Generated with Claude Code

Co-Authored-By: Claude <[email protected]>
@f-yoshihara
Copy link
Contributor Author

📝 「気になること」に他の Issue や PR との関連がある場合に、それを書いてもらうことを促すコメントを追加したい

@f-yoshihara
Copy link
Contributor Author

📝 「気になること」に他の Issue や PR との関連がある場合に、それを書いてもらうことを促すコメントを追加したい

気になっていること に追加するのは不自然に感じたので、その他の項目を設けてそこに上記のコメントを記載しました

@@ -0,0 +1,25 @@
## やったこと
<!-- 何がどう変わったのかを説明してください。必要に応じてスクリーンショットや動画も利用し、レビュワーが簡単に理解できるように工夫してください。 -->
Copy link
Contributor Author

Choose a reason for hiding this comment

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

レビュワーが簡単に理解できるように工夫してください

かと言って PR 作成に手間がかかりすぎないように(なるべく簡単にレビュー依頼できるように)誘導するコメントもあった方がいいかも?

@@ -0,0 +1,25 @@
## やったこと
<!-- 何がどう変わったのかを説明してください。必要に応じてスクリーンショットや動画も利用し、レビュワーが簡単に理解できるように工夫してください。 -->
Copy link
Contributor Author

Choose a reason for hiding this comment

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

PRの変更によってユーザーにとってどんなメリットがあるか?も記載してもらっても良いかも?

@f-yoshihara f-yoshihara self-assigned this Dec 24, 2025
@f-yoshihara f-yoshihara marked this pull request as ready for review December 24, 2025 08:16
@f-yoshihara f-yoshihara requested review from akira888 and nay December 24, 2025 08:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants