Skip to content

[Chore] 격리 환경 테스트를 통과해야 main 과 develop 에 머지되게 한다 - #334

Merged
KiSeungMin merged 1 commit into
developfrom
chore/require-test-before-merge
Sep 21, 2026
Merged

KiSeungMin merged 1 commit into
developfrom
chore/require-test-before-merge

Conversation

@KiSeungMin

Copy link
Copy Markdown
Contributor

✔️ 연관 이슈

📝 작업 내용

  • main 과 develop 에 branch protection 을 걸었습니다. 코드가 아니라 레포 설정이라 이 PR 머지와 상관없이 이미 적용되어 있습니다.
    • PR 필수, 격리 환경 테스트 required check, 리뷰 승인은 요구하지 않음
    • 관리자는 우회 가능(긴급 핫픽스용), force push 와 브랜치 삭제는 막음
  • test.yml 의 틀린 주석을 고쳤습니다. ci-prod 를 수동 실행이라고 적어 뒀는데, 실제로는 main push 로 바로 배포를 시작하고 이 워크플로를 기다리지 않습니다(ci-prod.yml:3-5, ci-prod.yml:69).

👤 사용자 영향

  • 없습니다. 레포 설정과 주석만 바뀝니다.

🔌 API 호환성

  • 요청/응답 DTO 변경 없음
  • 기존 Flutter 앱 버전과 호환 확인
  • breaking change 있음

🗄️ DB migration

  • 없음

🔐 인증/권한

  • 영향 없음

✅ 검증 결과

  • gh api repos/AI-SIP/OnO_BACKEND/branches/{main,develop}/protection 으로 다시 조회해서 required check 가 격리 환경 테스트(GitHub Actions), 관리자 강제 꺼짐, force push 꺼짐으로 들어간 것을 확인했습니다.
  • 이 PR 이 첫 적용 대상이라, 테스트가 끝나기 전에 머지 버튼이 막히는지 여기서 확인할 수 있습니다.

🚀 배포 리스크

  • develop 과 main 에 직접 push 가 막힙니다. 관리자 계정은 예외입니다.
  • 머지가 테스트 시간(약 5~6분)만큼 늦어집니다. 배포 워크플로는 바꾸지 않아서 배포 자체가 느려지지는 않습니다.

↩️ 롤백/대응 방법

  • 레포 Settings → Branches 에서 main, develop 규칙을 지우면 이전과 같아집니다. 주석은 revert 로 되돌아갑니다.

- ci-prod 는 수동 실행이 아니라 main push 로 바로 배포를 시작하고, 이 워크플로를 기다리지 않는다
- 그래서 배포를 막는 것은 PR 단계이고, main 과 develop 의 branch protection 이 격리 환경 테스트를 required check 로 건다
@KiSeungMin KiSeungMin added the chore 코드 수정, 내부 파일 수정 label Sep 21, 2026
@KiSeungMin KiSeungMin self-assigned this Sep 21, 2026
@KiSeungMin
KiSeungMin merged commit 288da42 into develop Sep 21, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

chore 코드 수정, 내부 파일 수정

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant