hotfix : deploy 문제 해결

[hiho1010]

Summary by CodeRabbit

• Chores
  • CI/CD 배포 워크플로우 설정 최적화
  • 배포 프로세스 스크립트 간소화
  • 워크플로우 포맷팅 개선

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

Caution

Review failed

The pull request is closed.

Walkthrough

GitHub Actions 워크플로우 설정이 업데이트되었습니다. cd-dev-deploy.yml에서 레지스트리 환경 변수가 제거되고, 배포 변수가 VERSION에서 IMAGE_TAG로 변경되었으며, APP_PORT가 추가되고, docker 로그인에서 sudo가 제거되었습니다. ci-pr.yml에서는 공백 줄이 정리되었습니다.

Changes

커뮤팀 / 파일(들)	변경 요약
워크플로우 환경 변수 리팩토링 .github/workflows/cd-dev-deploy.yml	REGISTRY, IMAGE_NAME 환경 변수 제거; VERSION을 IMAGE_TAG로 변경; APP_PORT 추가; 배포 단계에 전달되는 환경 변수 목록 업데이트
배포 스크립트 단순화 .github/workflows/cd-dev-deploy.yml	docker 로그인에서 sudo 제거; 쉘 스크립트 플래그를 set -euo pipefail에서 set -e로 변경
워크플로우 포맷팅 .github/workflows/ci-pr.yml	불필요한 공백 줄 제거; 동작 변화 없음

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

• 환경 변수 이름 변경의 일관성 검증 필요 (VERSION → IMAGE_TAG)
• set -euo pipefail에서 set -e로의 변경이 배포 스크립트의 오류 처리에 미치는 영향 확인 권장

Possibly related PRs

• Feat : dev 분리해서 배포, slack 알림 추가 #9: 동일하게 CI/CD 배포 워크플로우를 수정하며, IMAGE_TAG 및 APP_PORT와 같은 배포 환경 변수가 배포 단계로 전달되는 방식을 변경합니다.

Poem

🐰 환경 변수를 정리하고,
sudo 없이 가볍게 로그인하네,
IMAGE_TAG로 이름 지었고,
APP_PORT 새로 담았으니,
배포는 더욱 깔끔하게! 🚀

✨ Finishing touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch hotfix/code-deploy

---

📜 Recent review details

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 64f530c and 313b3b6.

📒 Files selected for processing (2)

• .github/workflows/cd-dev-deploy.yml (1 hunks)
• .github/workflows/ci-pr.yml (1 hunks)

---

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.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

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