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

[SP2] CI 스크립트에 빌드 step 추가 #213

Merged
merged 1 commit into from
Oct 12, 2023

Conversation

SeojinSeojin
Copy link
Member

@SeojinSeojin SeojinSeojin commented Oct 10, 2023

Summary

CI 스크립트란 해당 PR이 머지 가능한지 확인하는 스크립트로, CI 스크립트에 적힌 step들을 통과하지 못하면 머지가 불가능합니다.

  • AS-IS : CI 스크립트에서 lint까지 검사합니다.
  • TO-BE: CI 스크립트에서 빌드가 잘 되는지도 봅니다.

두번째 커밋만 봐주세요!!!!!

Screenshot

빌드 안 됨 빌드 잘 됨
스크린샷 2023-10-10 오후 1 10 07 스크린샷 2023-10-10 오후 1 15 03

Comment

시간

다 돌아가는데 최대 2분정도 걸릴 것 같아용 ..
너무 급하면 그냥 머지해도 될 것 같습니다!!

환경변수

스크린샷 2023-10-10 오후 1 09 22

이 과정에서 repository secret에 저희 환경변수값들을 집어넣어 두었습니다!
앰플리튜드 키는 혜준언니 테스트키를 넣었고, api url은 프로덕션으로 넣어두었어요

Copy link
Contributor

@f0rever0 f0rever0 left a comment

Choose a reason for hiding this comment

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

오호 이렇게 CI 추가하는건 처음봤는데 신기하네요! 감사합니당!

@SeojinSeojin
Copy link
Member Author

우영언니 기존엔 어떤 식으로 CI CD 써보았는지 말해줄 수 있나용?! 혜준언니도!!!

@f0rever0
Copy link
Contributor

빌드를 직접 수정?해본적이 없어가지구...ㅎㅎ 이렇게 커스텀하는 구나! 신기했어요!!

Copy link
Member

@solar3070 solar3070 left a comment

Choose a reason for hiding this comment

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

빌드 에러를 겪고 나서 고치는 것에 멈추지 않고 자동화 코드까지 짜셨군요! 정말 리스펙합니다 🚀
저는 깃허브 액션은 API 키를 감출 필요가 있을 때만 사용해봤습습니다. 따로 CI/CD 구축을 해본 적은 없어요.

나중에 배포 환경을 클라우드 플레어로 바꿀 때 환경 변수를 제외한 부분도 코드 수정이 필요하게 될지가 궁금하네요..!

@SeojinSeojin
Copy link
Member Author

나중에 해보면 좋을 일들 찾아서 한번씩 깃헙액션 코드 짜보는 테스크 해봐도 재밌을 것 같네요!!!
요것은 깃허브에서 머지 가능 여부 확인하기 위해 돌리는 스크립트라 배포와는 관련이 없을 것이에요..!!
클라우드플레어 배포에서 만약 .env에 있는 변수들을 깃허브에서 가져오게 된다면 요 수정사항과 관련이 있게 될 것입니다!

@SeojinSeojin SeojinSeojin force-pushed the fix/#212_ci-check-build branch from fe56e66 to 90e7a95 Compare October 12, 2023 01:06
@SeojinSeojin SeojinSeojin merged commit 2af72bf into develop Oct 12, 2023
1 check passed
@SeojinSeojin SeojinSeojin deleted the fix/#212_ci-check-build branch October 12, 2023 01:11
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.

CI 머지 조건에 build 잘 되는지도 추가하기
3 participants