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

[BE] chore: release cd를 돌릴 수 있는 브랜치 한정 #1004

Merged
merged 1 commit into from
Dec 15, 2024

Conversation

nayonsoso
Copy link
Contributor


🚀 어떤 기능을 구현했나요 ?

  • release cd 를 돌릴 수 있는 브랜치를 'release'와 'hotfix/'로 시작하는 브랜치로 한정했습니다.
  • 위 이슈를 반복하지 않기 위함입니다.

🔥 어떻게 해결했나요 ?

  • 'release'가 아니거나 'hotfix/'로 시작하지 않으면 워크플로우를 즉시 중단하게 했습니다.

📝 어떤 부분에 집중해서 리뷰해야 할까요?

  • 브랜치 이름을 이렇게 workflow에 박아두면, 장단점이 있습니다.
    • 장점 : prod 서버가 이상한 브랜치에서 돌지 않게 지킬 수 있다.
    • 단점 : 브랜치 이름이 하드코딩으로 들어간다.
  • 워크플로우 파일에 브랜치 이름이 들어가는 경우는 흔하니, 단점이 치명적이지 않다 생각해서 이렇게 PR 올립니다.

📚 참고 자료, 할 말

  • dev cd 파일도 이렇게 수정해야 할까요? 아니면 dev cd 는 조금 더 자유도를 둘까요?

Copy link
Contributor

@skylar1220 skylar1220 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  1. 하드 코딩으로 들어가는 부분은 치명적이지 않다 동의합니다!
  2. dev는 테스트 목적으로 다양한 상황에서 cd를 돌리니 자유도를 두는 게 좋다고 생각해요~

이슈 방지를 위해 제도 도입이 적절한지를 생각해보았는데,
인지 부하가 커지는 부분이 없고, 오히려 휴먼 에러를 방지할 수 있어서 좋다고 생각이 드네요👍👍

@nayonsoso nayonsoso merged commit b7b88ac into develop Dec 15, 2024
5 checks passed
@donghoony donghoony deleted the be/chore/limit-release-cd-branch branch December 18, 2024 11:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants