ci: remove un-active reviewers #199
Conversation
froggy1014
requested review from
SEMIN-97,
bae-sh,
developerjhp,
heeji289,
hy57in,
jiji-hoon96,
noahluftyang,
synuns and
yeojini
as code owners
|
✅ Deploy Preview for side-design-system ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
Walkthrough.gitHub/CODEOWNERS의 소유자 목록이 갱신되어 일부 계정이 제거되고 신규 계정이 추가되었습니다. 코드나 런타임 동작 변경은 없습니다. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Possibly related PRs
Suggested labels
Suggested reviewers
Poem
Tip 🔌 Remote MCP (Model Context Protocol) integration is now available!Pro plan users can now connect to remote MCP servers from the Integrations page. Connect with popular remote MCPs such as Notion and Linear to add more context to your reviews and chats. ✨ Finishing Touches🧪 Generate unit tests
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. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. CodeRabbit Commands (Invoked using PR/Issue comments)Type Other keywords and placeholders
CodeRabbit Configuration File (
|
Codecov Report✅ All modified and coverable lines are covered by tests. 🚀 New features to boost your workflow:
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (2)
.github/CODEOWNERS (2)
1-1: 유지보수성 향상을 위해 개인 계정 대신(또는 함께) 팀 핸들(@org/team) 사용 검토개인 계정 기반은 변동(퇴사/휴면)에 취약합니다. 조직 팀을 사용하면 관리 비용을 줄이고 지속성을 확보할 수 있습니다.
예시 1) 팀으로 대체:
-* @kimdaeyeobbb @noahluftyang @froggy1014 @synuns @jiji-hoon96 @haedawn +* @your-org/reviewers예시 2) 팀을 우선으로, 개인은 백업:
-* @kimdaeyeobbb @noahluftyang @froggy1014 @synuns @jiji-hoon96 @haedawn +* @your-org/reviewers @kimdaeyeobbb @noahluftyang @froggy1014 @synuns @jiji-hoon96 @haedawn원하시면 실제 조직/팀 핸들에 맞춘 최종 라인을 제안드리겠습니다.
1-1: 변경 이력 주석(메타데이터) 추가 제안후속 추적을 위해 변경 이유/시점을 주석으로 남기면 좋습니다. 필수는 아니지만 향후 감사/회고에 유용합니다.
+# 기본 코드 오너 (2025-08-21): 비활성 리뷰어 제거, @haedawn 추가 * @kimdaeyeobbb @noahluftyang @froggy1014 @synuns @jiji-hoon96 @haedawn
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
💡 Knowledge Base configuration:
- MCP integration is disabled by default for public repositories
- Jira integration is disabled by default for public repositories
- Linear integration is disabled by default for public repositories
You can enable these sources in your CodeRabbit configuration.
📒 Files selected for processing (1)
.github/CODEOWNERS(1 hunks)
🔇 Additional comments (3)
.github/CODEOWNERS (3)
1-1: PR 의도 부합: 비활성 리뷰어 제거 및 기본 CODEOWNERS 정리 완료리뷰어 풀을 슬림화하려는 목적과 일치하며, 런타임/빌드 영향은 없습니다. 기본 경로
*에 대해 현재 활성 멤버만 남겨 깔끔합니다.
1-1: 브랜치 보호 및 CODEOWNERS 리뷰 요구 설정 확인 요청현재 제공된 스크립트는 로컬에 Git 리포지토리 정보가 없어서 자동 실행이 불가능합니다. 다음 수정된 명령어를 GitHub CLI가 올바르게 동작하는 환경(로컬 터미널 또는 CI)에서 직접 실행해 주세요.
#!/bin/bash set -euo pipefail # owner/repo 형태로 리포지토리명을 직접 지정 REPO="your-org/your-repo" # 기본 브랜치명을 직접 지정 (예: main, master) DEFAULT_BRANCH="main" echo "[브랜치 보호] require_code_owner_reviews 설정 확인" gh api -H "Accept: application/vnd.github+json" \ "/repos/$REPO/branches/$DEFAULT_BRANCH/protection" \ | jq '.required_pull_request_reviews.require_code_owner_reviews' echo echo "[CODEOWNERS] 지정된 오너들의 권한 확인" gh api -H "Accept: application/vnd.github+json" \ "/repos/$REPO/contents/.github/CODEOWNERS" \ | jq -r '.content' | base64 --decode \ | awk 'NF && $1=="*" {for(i=2;i<=NF;i++) print $i}' \ | tr -d '@' \ | while read -r user; do perm=$(gh api -H "Accept: application/vnd.github+json" \ "/repos/$REPO/collaborators/$user/permission" \ -q .permission 2>/dev/null || echo "none") echo "$user: ${perm:-not a collaborator}" done위 스크립트를 실행한 후 출력 결과를 공유해 주시면, 설정이 올바르게 적용되었는지 확인하고 다음 절차를 안내드리겠습니다.
1-1: 중복 CODEOWNERS 및 규칙 충돌 점검 완료 — 추가 조치 불필요
fd -HI --glob CODEOWNERS결과: 리포지토리에.github/CODEOWNERS파일 하나만 존재 (중복 없음)- 상위 50줄 확인 결과: 단일 매칭 규칙(
* @kimdaeyeobbb @noahluftyang @froggy1014 @synuns @jiji-hoon96 @haedawn)만 정의되어 있어 충돌 우려 없음
Changes
전 기수분들 전부 리뷰어 할당하는거 지울게요 ~
Summary by CodeRabbit