Skip to content

docs: align required checks with stable gates#54

Merged
seonghobae merged 1 commit into
developfrom
docs/github-required-checks-adjustment
Mar 12, 2026
Merged

docs: align required checks with stable gates#54
seonghobae merged 1 commit into
developfrom
docs/github-required-checks-adjustment

Conversation

@seonghobae

Copy link
Copy Markdown
Collaborator

Summary

  • update required-check docs to match the stable branch protection gate set after removing the flaky external CodeRabbit status context
  • keep CodeRabbit as the requested AI review path while documenting why the hosted status context is no longer a required branch-protection check

Verification

  • python3 scripts/checks/verify_github_bootstrap_policy.py
  • python3 scripts/checks/verify_docs.py

@coderabbitai

coderabbitai Bot commented Mar 12, 2026

Copy link
Copy Markdown
Contributor

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: 67c132b1-b70f-4b7d-b5b6-44acd25798ff

📥 Commits

Reviewing files that changed from the base of the PR and between f60b4ef and 09f15c3.

📒 Files selected for processing (3)
  • docs/security/code-security.md
  • docs/security/github-required-checks.md
  • docs/workflow/github-bootstrap-execution-policy.md

Cache: Disabled due to Reviews > Disable Cache setting

Disabled knowledge base sources:

  • Linear integration is disabled

You can enable these sources in your CodeRabbit configuration.


📝 Walkthrough

Summary by CodeRabbit

릴리스 노트

Documentation

  • 보안 정책 문서 업데이트: 외부 AI 검토 상태 컨텍스트에 대한 실행 지침 추가 및 단독 게이트로 사용하지 않도록 명시
  • GitHub 필수 검사 구성 변경: 외부 검토 도구의 요구를 제거하고 검토 요청 및 정책 기반 접근으로 전환
  • 워크플로우 실행 정책 수정: 저장소 소유의 안정된 검사 우선 적용 및 외부 검토 결과 기록 절차 추가

Walkthrough

CodeRabbit 상태 검사 요구를 필수에서 선택적 검토 요청 정책으로 변경하고, 외부 AI 검토 상태(context)를 단독 필수 게이트로 사용하지 말라는 운영상 enforcement note를 문서들에 추가했습니다.

Changes

Cohort / File(s) Summary
보안 문서: 코드 보안 정책
docs/security/code-security.md
외부 AI 리뷰 상태가 단독 필수 게이트가 되어선 안 된다는 enforcement 문구 추가(운영상 불안정성 관련).
보안 문서: GitHub 필수 체크
docs/security/github-required-checks.md
CodeRabbit을 required 체크에서 제거하고, CodeRabbit 리뷰 요청 및 리뷰-등가 정책으로 변경. 저장소 소유의 안정적 체크를 선호하는 enforcement note 추가.
워크플로우: 부트스트랩 실행 정책
docs/workflow/github-bootstrap-execution-policy.md
브랜치 보호의 필수 체크에서 CodeRabbit 제거. PR 워크플로우에 CodeRabbit 리뷰를 요청하고 응답이 정상일 때 결과를 기록하는 항목 추가.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

🐰 깡총깡총 뛰며 말하네,
외부별이 반짝여도 전부는 아니네.
요청으로 부르면 와서 도와주고,
저장소는 튼튼히 지키리라.
작은 발자국이 큰 변화를 만든다네. 🥕✨

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch docs/github-required-checks-adjustment

Comment @coderabbitai help to get the list of available commands and usage tips.

@seonghobae

Copy link
Copy Markdown
Collaborator Author

@coderabbitai review

@seonghobae seonghobae enabled auto-merge (squash) March 12, 2026 05:49
@coderabbitai

coderabbitai Bot commented Mar 12, 2026

Copy link
Copy Markdown
Contributor
✅ Actions performed

Review triggered.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

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