Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Warning Rate limit exceeded
⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. Walkthrough이번 변경사항은 GitHub Actions 워크플로우 및 빌드 설정을 정리하고 개선하는 세 가지 주요 수정을 포함합니다:
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~25 분 Suggested reviewers
🚥 Pre-merge checks | ✅ 2 | ❌ 2❌ Failed checks (1 warning, 1 inconclusive)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
There was a problem hiding this comment.
🧹 Nitpick comments (2)
.github/workflows/release.yml (2)
35-35:actions/checkout버전이 일관되지 않아요.Line 35에서는
actions/checkout@v3을, Line 53에서는actions/checkout@v4를 사용하고 있어요. 동작에 문제가 생기진 않지만, 일관성을 위해v4로 통일하는 것을 권장드려요.♻️ 수정 제안
- - uses: actions/checkout@v3 + - uses: actions/checkout@v4
89-134: 브랜치별 승격 루프 로직이 잘 구성되어 있어요 👍전체 흐름을 정리하면:
- 브랜치 미존재 →
main에서 새로 생성- SHA 동일 +
force_redeploy→ empty commit으로 배포 트리거- SHA 동일 +
force_redeploy아님 → skip- SHA 상이 → ancestry 검증 후 업데이트
- ancestry 실패 → 즉시 중단 (
exit 1)한 가지 참고 사항이 있어요.
target=both일 때 첫 번째 브랜치에서 ancestry 검사가 실패하면exit 1로 스크립트가 종료되어 두 번째 브랜치는 처리되지 않아요. 의도된 fail-fast 전략이라면 괜찮지만, 두 브랜치를 독립적으로 처리하고 싶다면 에러를 누적한 뒤 마지막에 종합 결과를 반환하는 방식도 고려해 볼 수 있어요.💡 독립 처리 방식 예시 (선택 사항)
+ FAILED=0 for BRANCH in $RELEASE_BRANCHES; do # ... (existing fetch/check logic) ... if ! git merge-base --is-ancestor "origin/$BRANCH" origin/main; then echo "$BRANCH is not an ancestor of main. Resolve release history before promotion." >&2 - exit 1 + { + echo "- $BRANCH: ❌ not an ancestor of main" + } >> "$GITHUB_STEP_SUMMARY" + FAILED=1 + continue fi # ... (existing push logic) ... done + + if [ "$FAILED" -ne 0 ]; then + echo "One or more branches failed promotion." >&2 + exit 1 + fi
97b508c to
cf87dd1
Compare
Summary
release.yml수동 실행 입력값에force_redeploy(boolean)를 추가했습니다.force_redeploy=true이면 대상 브랜치에 empty commit을 생성해 Vercel 배포 트리거를 강제로 발생시키도록 구현했습니다.target(web/admin/both) 분기 로직은 유지하면서 step summary에 강제 트리거 결과를 기록하도록 확장했습니다.