Skip to content

Conversation

@sxunea
Copy link
Contributor

@sxunea sxunea commented Oct 27, 2024

📝 Work Description

  • develop / main 에 PR 올릴 시 트리거
    • cd 테스트를 위해서 develop 트리거 한거고, 어푸르브 되면 수정해서 main으로의 PR에만 트리거하겠습니다
  • keystore, key, pw 등은 github secrets 가져와 적용
  • PR 제목에서 release v*.. 형식의 버전을 추출
  • GitHub Actions Artifacts에 릴리즈 apk 추출
  • Firebase App Distribution 테스터들에게 apk 제공

📣 To Reviewers

  • apk 정상 추출 시 슬랙 알림 / crashlytics 오류 보고 자동화 등의 스크립트를 추가할 수 있습니다 !
    • 성공 슬랙 알림은 깃허브에서 release published로 이미 슬랙 알림 주고 있어서 따로 안추가했어요 ! 원하면 추가해두겠습니다.
    • crashlytics 오류 보고는 하은이가 crashlytics 추가해두면 스크립트 작성해둘게용
  • CD 트리거를 PR제목이 버전형식일때로 설정도 가능해요 ! 지금 방식(main으로의 PR) 이나 버전제목 둘 중 어떤게 좋은지 의견 주세요
  • 리버트 된 커밋은 메시지를 잘못 설정해서 리버트한거라 그냥 file changed 보는게 편합니다
image

@sxunea sxunea added the add ➕ feat 이외의 부수적인 코드, 파일, 라이브러리 추가 label Oct 27, 2024
@sxunea sxunea requested a review from leeeha October 27, 2024 06:38
@sxunea sxunea self-assigned this Oct 27, 2024
@@ -0,0 +1,102 @@
# JVM 옵션을 설정, 최대 힙 메모리를 4GB로
Copy link
Member

Choose a reason for hiding this comment

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

4GB로 설정한 이유가 있는지 궁금합니다!

pull_request:
branches:
- develop
- main
Copy link
Member

Choose a reason for hiding this comment

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

현재 올린 코드가 프로덕션 앱 배포까지 자동화 하는 건지, 테스트 앱 배포만 자동화 하는 건지 궁금합니다!

저는 테스트 앱 배포 자동화라고 이해했는데, 그렇다면 release 브랜치에 PR이 올라올 때 CD가 트리거 되어야 할 거 같아요!

(main 브랜치는 프로덕션 버전 앱 배포 준비할 때 사용하고 있으니까)

@leeeha
Copy link
Member

leeeha commented Oct 31, 2024

@sxunea 성공 슬랙 알림은 깃허브에서 release published로 이미 슬랙 알림 주고 있어서 따로 안추가했어요 !

-> 이거는 yml 파일에서 못 본 거 같은데, 어떻게 슬랙에 알림이 뜨는 건지 궁금하네요!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

add ➕ feat 이외의 부수적인 코드, 파일, 라이브러리 추가 혜선 🐱

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

[add] Github Actions CD 적용

3 participants