Skip to content

docs: PRテンプレ例のCloses表記を調整#194

Open
ootakazuhiko wants to merge 2 commits intomainfrom
fix/pr-template-closes-example-20260214
Open

docs: PRテンプレ例のCloses表記を調整#194
ootakazuhiko wants to merge 2 commits intomainfrom
fix/pr-template-closes-example-20260214

Conversation

@ootakazuhiko
Copy link
Collaborator

変更内容

  • PRテンプレート例の Closes #xxxx を、例として分かりやすい Closes #123 に変更しました。

背景

  • xxxx のままだとプレースホルダに見え、読者に「このまま使って良いのか」が伝わりにくいため。

影響

  • 表示/体裁の調整のみで、テンプレートの意図は変更しません。

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

PRテンプレート例の「Closes」表記を、プレースホルダに見えない具体例へ置き換えて、読者が意図を理解しやすくするためのドキュメント調整です。

Changes:

  • PRテンプレート例の Closes #xxxxCloses #123 に変更

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.


## 受入条件(Issue)
- Closes #xxxx
- Closes #123
Copy link

Copilot AI Feb 14, 2026

Choose a reason for hiding this comment

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

この例は「このまま使える」テンプレート例として示されているため、Closes #123 をそのままコピペすると実在する Issue/PR #123 を参照(場合によってはクローズ)してしまう可能性があります。Closes #123(例) のように例である旨を併記するか、Closes #<issue-number> 等の“置き換え必須”表現にして誤操作を避けたいです。

Suggested change
- Closes #123
- Closes #<issue-number>

Copilot uses AI. Check for mistakes.
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

ご指摘ありがとうございます。

例を Closes #<issue-number> に変更し、実在の Issue を参照してしまうリスクを避けました。

@ootakazuhiko
Copy link
Collaborator Author

Copilot のインライン指摘に対応しました。

  • Closes #123Closes #<issue-number> に変更(誤参照/誤クローズのリスク回避)
  • CI: qa / docs-forbidden ともに PASS

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.

1 participant