Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: 조건문의 브랜치 변경 #335

Merged
merged 1 commit into from
Jun 2, 2024

Conversation

HumanIearning
Copy link
Collaborator

💡 배경 및 개요

PR을 하게 된 문제상황, 배경 및 개요에 대해서 작성해주세요!

퍼블리싱의 경우 스크린샷/동영상도 추가해주면 좋아요!

Resolves: #334

📃 작업내용

원래 master-cd에서 develop까지 cd하려했는데 develop cd가 따로 있길래
develop cd사용할 수 있게끔 조건문 브랜치만 변경해줌

🙋‍♂️ 리뷰노트

develop에 merge(push)가 많이 일어날 경우 ecr에 과거 버전 이미지가 쌓여서 비용이 지불될 수 있으니 주기적으로 지워줘야함

✅ PR 체크리스트

템플릿 체크리스트 말고도 추가적으로 필요한 체크리스트는 추가해주세요!

  • 이 작업으로 인해 변경이 필요한 문서가 변경되었나요? (e.g. .env, 노션, README)
  • 이 작업을 하고나서 공유해야할 팀원들에게 공유되었나요? (e.g. "API 개발 완료됐어요", "환경값 추가되었어요")
  • 작업한 코드가 정상적으로 동작하나요?
  • Merge 대상 브랜치가 올바른가요?
  • PR과 관련 없는 작업이 있지는 않나요?

🎸 기타

@HumanIearning HumanIearning merged commit 71e50e1 into develop Jun 2, 2024
1 check passed
@HumanIearning HumanIearning deleted the feature/334/cd-image-tag branch June 2, 2024 08:30
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.

CD에서 브랜치에 따라 이미지의 태그를 변경하게 합니다
1 participant