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

git action 으로 cicd 구축 #41

Merged
merged 26 commits into from
Nov 13, 2024
Merged

git action 으로 cicd 구축 #41

merged 26 commits into from
Nov 13, 2024

Conversation

mjh000526
Copy link
Collaborator

@mjh000526 mjh000526 commented Nov 12, 2024

#️⃣연관된 이슈

#35

📝작업 내용

  1. main브랜치를 향한 pr,merge마다 CI를 통해 백엔드코드 build
  2. release로 push할때 CD를 통해 도커 이미지 및 배포
  3. CI는 line, turbo로 build
  4. PR에 "💻 Be" 태그가 없으면 CI skip

스크린샷 (선택)

image

💬리뷰 요구사항(선택)

CI는 pnpm캐싱을 이용해서 시간을 많이 단축했습니다. CD과정은 추후에 최적화가 필요하다 판단되면 하는게 좋을 것 같습니다.
궁금한 설정이 있다면 리뷰남겨주세요. 코멘트뿐 아니라 개발위키에도 상세히 추가하겠습니다.

@ATeals
Copy link
Collaborator

ATeals commented Nov 12, 2024

지금은 프론트엔드 코드가 릴리즈 될때도 백엔드 ci/cd가 돌아갈거 같습니다. 그렇다면 ci/cd 체킹하는 시간이 너무 길거 같은데, 모노레포 설정을 통해서 ci/cd 경로를 체킹하는 식의 방법이 필요할 것 같아요!

@naarang
Copy link
Collaborator

naarang commented Nov 12, 2024

지금은 프론트엔드 코드가 릴리즈 될때도 백엔드 ci/cd가 돌아갈거 같습니다. 그렇다면 ci/cd 체킹하는 시간이 너무 길거 같은데, 모노레포 설정을 통해서 ci/cd 경로를 체킹하는 식의 방법이 필요할 것 같아요!

https://f-lab.kr/blog/wise-use-of-github-actions-in-monorepo

변경된 부분에 대해서 ci/cd를 진행하는 거 저번주에 찾아보다가 이런 블로그 본 적 있어요! 도움이 될 지는 모르겠습니다ㅠ

@mjh000526
Copy link
Collaborator Author

좋은 레퍼런스 감사합니다. file diff 체킹과 병렬처리를 통해서 단축할 수 있을 것 같네요. FE 배포에 대해 얘기해보고 바로 적용해보면 좋을 것 같아요

Copy link
Collaborator

@naarang naarang left a comment

Choose a reason for hiding this comment

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

수고하셨습니다~!!

@mjh000526 mjh000526 merged commit e955afe into main Nov 13, 2024
1 check passed
@mjh000526 mjh000526 deleted the feature-#35-BE-cicd branch November 13, 2024 01:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants