docs: ADR 0002(取り込み)を承認済みにする by yokinist · Pull Request #24 · codeforjapan/code4security · GitHub
Skip to content

docs: ADR 0002(取り込み)を承認済みにする - #24

Open
yokinist wants to merge 1 commit into
codeforjapan:mainfrom
yokinist:docs/adr-0002-approve
Open

yokinist wants to merge 1 commit into
codeforjapan:mainfrom
yokinist:docs/adr-0002-approve

Conversation

@yokinist

Copy link
Copy Markdown
Collaborator

ADR 0002(first-security-checkup と weekly-security-checker を本体に取り込む)を承認済みにします。

ADR 0002 は「yokinist さんの許可をもらってから承認済みにし、取り込みを実行する」としていました。作者の yokinist が、許可として自分でこの PR を出します。取り込みはこの PR が承認されてから、別の PR で行います。

変えたこと

  • ステータスを承認済みにし、「決定の記録」を足した。承認者と承認日は、レビューのあとに書き足す
  • 「yokinist さんと決める」としていた 3 点に、作者の答えを書いた
  • 懸念点の「元の checker をすでに使っている人がいる」を事実に合わせた。2026-10-22 の時点で、公開リポジトリで使っているところは見つからず、作者も非公開のリポジトリで使っていない
  • 今後の方向性に、取り込みのあとに残る作業を書いた
    • マージは履歴が残る「Create a merge commit」で行う
    • GitHub Pages からの公開とリンクの付け替え
    • guide/・checker/ の CONTRIBUTING・SECURITY などを、リポジトリ直下の文書とまとめる

@yu23ki14 レビューをお願いします。ADR 0010 に沿って、48 時間反応がなければ承認として進めます。

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@yokinist
yokinist requested a review from yu23ki14 October 21, 2026 16:25
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