Skip to content

GitHub の PR テンプレートを導入する #96

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

Open
f-yoshihara wants to merge 6 commits into
base: master
Choose a base branch
from
Open

GitHub の PR テンプレートを導入する #96

Changes from all commits
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
2df2e0c
PR テンプレートを追加
f-yoshihara Nov 17, 2025
3b32e8e
PR テンプレートの「やっていないこと」セクションを修正
f-yoshihara Nov 17, 2025
43288e8
PR テンプレートの項目追加・修正とコメント修正
f-yoshihara Dec 24, 2025
dc32c85
PRテンプレートに概要記載のための項目追加
f-yoshihara Jan 21, 2026
6a1fc89
PRテンプレートの動作確認手順欄の構造変更
f-yoshihara Jan 21, 2026
b748e79
動作確認実施状況についてのコメント修正
f-yoshihara Feb 18, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
34 changes: 34 additions & 0 deletions .github/pull_request_template.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,34 @@
## 対応 Issue
<!-- 対応するIssueがあれば記載してください。なければ項目ごと削除してください -->
<!-- 以下のように記載することでPRとIssueをリンクさせることができます。ref: https://docs.github.com/ja/issues/tracking-your-work-with-issues/using-issues/linking-a-pull-request-to-an-issue#linking-a-pull-request-to-an-issue-using-a-keyword -->
<!-- closes #10 -->


## このPRがマージされるとうれしい理由
<!-- このPRによってユーザーにとってどんな利益があるか、あるいはどんな課題が解決されるか等を記載してください。 -->

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

@f-yoshihara f-yoshihara Dec 24, 2025

Choose a reason for hiding this comment

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

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

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

Copy link
Contributor

@akira888 akira888 Feb 18, 2026

Choose a reason for hiding this comment

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

伝える側が努力したほうが伝わるコストは低くなるとは思いますが、そんな詳細に説明しなくていいよ〜ということもありますよね…
塩梅については万葉道場で学んでもらうとかでもいいのかもと思いました

Copy link
Contributor Author

@f-yoshihara f-yoshihara Dec 24, 2025

Choose a reason for hiding this comment

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

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


## やっていないこと
<!-- このPRで、対応した方が良い・対応しても良いが、あえてやっていないことがあればその理由を記載してください（例: XXは後続の Issue #10 で対応予定です）。特にない場合は「なし」と記載 -->

## 気になっていること
<!-- 不安な点や相談したい点、よく見てもらいたい点などがあれば記載してください。特にない場合は「なし」と記載 -->

## その他
<!-- その他共有事項など自由に記載してください -->
<!-- 他のPRと関連がある場合などもこちらに記載してください（例: PR #11 の後続です） -->

## 動作確認
<!-- レビュアー（このアプリの仕様を知らないエンジニアを想定してください）が迷わず実施できるように、以下の項目は具体的に記載してください。 -->

### 前準備
<!-- 前準備が必要な場合は記載してください。 -->

### PRの変更内容の確認手順
<!-- アプリの操作手順と期待される動作を具体的に記載してください。 -->

### 実施状況
<!-- 動作確認の実施状況を明示してください（特別な事情がない限り、自身で動作確認を行ってからレビュー依頼をするようにしてください）。-->
- [ ] ローカル開発環境での動作確認を実施済み