Skip to content

Comments

Init(client): Auto Build Github Action#31

Merged
gwagjiug merged 9 commits intodevelopfrom
init/build-workflow/#15
Jul 3, 2025
Merged

Init(client): Auto Build Github Action#31
gwagjiug merged 9 commits intodevelopfrom
init/build-workflow/#15

Conversation

@minjeoong
Copy link
Member

@minjeoong minjeoong commented Jul 3, 2025

📌 Summary

이 PR은 GitHub Actions 기반 Auto Build를 설정하여 PR 생성 시 자동 빌드 검증을 수행할 수 있도록 합니다.

📚 Tasks

Jenkins 로 배포 CI / CD 를 구축했지만, 해당 PR 을 올리는 이유는
jenkins 안에서의 build 와 와 github build 의 역할이 다르기 때문입니다.

jenkins 의 build 역할과 한계

  • 현재 Jenkins는 main 브랜치에 병합되었을 때만 pnpm build를 실행하여 배포 전 빌드 오류를 감지합니다.
  • 하지만 Jenkins는 로컬 브랜치나 PR에 대해 빌드 체크를 수행하지 않기 때문에, 다음과 같은 문제가 발생합니다
  1. 로컬에서 빌드 테스트를 하지 않고 PR을 올릴 경우, Jenkins에선 감지되지 않음
  2. 최종적으로 main에 병합되기 직전까지는 빌드 실패 여부를 자동으로 확인할 수 없음 (로컬 빌드를 꼭 해야함)

그렇다면 Build Github Action 은 무슨 역할?

  • GitHub Actions는 모든 PR에 대해 자동으로 빌드 파이프라인을 실행합니다.
  • PR 생성 또는 커밋 시점에 자동으로 pnpm build 실행합니다.
  • 빌드 실패 시 머지를 차단합니다. (즉, 선제적 오류 방지)
  • 코드 리뷰 시 빌드 가능 여부를 검증할 수 있습니다.

따라서 Jenkins는 배포 전용, GitHub Actions는 PR 품질 검증 전용으로 역할이 분리됩니다.

해당 PR 이 머지 되었을 때 develop 트래킹 잘하는지 jenkins 배포 체크 하도록 하겠습니다

이를 위한 테스트 코드 하나 더 추가되었습니다

@minjeoong minjeoong self-assigned this Jul 3, 2025
@minjeoong minjeoong requested a review from a team as a code owner July 3, 2025 13:38
@minjeoong minjeoong requested review from 1jiwoo27, gwagjiug, hansoojeongsj, jeonghoon11 and jogpfls and removed request for a team July 3, 2025 13:38
Copy link
Member

@gwagjiug gwagjiug left a comment

Choose a reason for hiding this comment

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

제가 5분컷 가능하다고 했죠???ㅎㅎㅎ 무한 신뢰 드립니다.. 이것으로 기본적인 cicd 구성이 어느정도 끝났네요 수고하셨습니다 👍

Copy link
Member

@hansoojeongsj hansoojeongsj left a comment

Choose a reason for hiding this comment

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

PR 잘 읽었습니다, 감사합니다 ㅎㅎ
설명도 명확하고 현재 Jenkins와 GitHub Actions의 역할 분리도 이해됐어요 :)
덕분에 자동 빌드 검증 체계가 훨씬 견고해질 것 같네요!

작성해주신 CI/CD 워크플로우, 정말 최고입니다 !!
깔끔하게 잘 짜주셔서 바로 적용해도 문제없을 것 같습니다 👍

@gwagjiug gwagjiug merged commit 7e685c5 into develop Jul 3, 2025
1 check passed
@gwagjiug gwagjiug deleted the init/build-workflow/#15 branch July 3, 2025 14:01
@minjeoong minjeoong linked an issue Jul 4, 2025 that may be closed by this pull request
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.

[Init]: cicd 및 배포 세팅

3 participants