Skip to content

docs: flatten the what else is affected section in the pull request template - #51

Merged
panicboat merged 1 commit into
mainfrom
docs/pr-template-flat-affected
Oct 31, 2026
Merged

panicboat merged 1 commit into
mainfrom
docs/pr-template-flat-affected

Conversation

@panicboat

Copy link
Copy Markdown
Owner

What This PR Does

PR テンプレートの What Else Is Affected から子セクションをなくし、1 つの節にまとめます。

What Needs to Change

  • What Else Is Affected が Documents と Systems & Operations に分かれています。
    • Reason: Documents のチェックボックスを判定に使っていた Docs Sync Check が、チェックを見なくなります。

What Changes for Users

  • PR を書く人が影響を書く場所
    • Before: 文書はチェックボックス、それ以外は Systems & Operations に分けて書きます。
    • After: What Else Is Affected に文書も含めてまとめて書きます。

What to Review Closely

なし

What Was Decided

  • 文書への影響は、チェックボックスをやめて書く対象の例に含めます。
    • 理由: 文書への影響を書き漏らさないようにするためです。
    • 間違っていた場合の影響: コメントの文面を直すだけです。

What Else Is Affected

  • この template を使うすべてのリポジトリの新しい PR に反映されます。
    • 開いている PR の本文は変わりません。
  • panicboat/monorepo の Docs Sync Check は、#1872 が merge されるまで ### Documents を照合します。
    • 節がない本文ではこの照合は skip されるので、merge の順番は問いません。
  • dotfiles の AGENTS.md にある「Documents 節を除く」が空振りになります。
    • この PR では変更しません。

How It Was Verified

  • git diff で変更が What Else Is Affected の中だけであることを確認しました。
  • テンプレートの変更なので、実行するテストはありません。

Related Issues and PRs

🤖 Generated with Claude Code

https://claude.ai/code/session_01N1XYRrfD84v1kxDKUBuPq8

…emplate

The documents checkboxes no longer drive the docs sync check, so nothing needs the section split into documents and systems.

Claude-Session: https://claude.ai/code/session_01N1XYRrfD84v1kxDKUBuPq8
Signed-off-by: panicboat <panicboat@gmail.com>
@coderabbitai

coderabbitai Bot commented Oct 31, 2026 •

Copy link
Copy Markdown

Important

  • Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 126b58bd-82b6-4d58-8f94-f30966f0d72c
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

Share

@panicboat
panicboat marked this pull request as ready for review October 31, 2026 10:37
@panicboat
panicboat merged commit 3300c4f into main Oct 31, 2026
7 checks passed
@panicboat
panicboat deleted the docs/pr-template-flat-affected branch October 31, 2026 10:38
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