Skip to content

[CHORE] CI 도입#7

Merged
i-meant-to-be merged 2 commits into
developfrom
chore/#5
Jun 26, 2026
Merged

[CHORE] CI 도입#7
i-meant-to-be merged 2 commits into
developfrom
chore/#5

Conversation

@i-meant-to-be

@i-meant-to-be i-meant-to-be commented Jun 25, 2026

Copy link
Copy Markdown
Collaborator

🚩 연관 이슈

closed #5

📝 작업 내용

CI를 도입합니다! 기본 골자는 어제 회의 내용과 동일합니다만, 몇 가지 추가로 안내할 사항이 있습니다.

추가 변경 사항

읽기만 허용

permissions:
  contents: read

코드 품질 검사가 목적인 만큼 쓰기 권한이 필요하지 않습니다. 위 설정을 추가하여 권한을 읽기로만 제한했습니다.

마지막 CI만 유지

concurrency:
  group: android-ci-${{ github.ref }}
  cancel-in-progress: true

이 설정은 같은 브랜치에 대한 CI 실행은 오직 하나만 유지하도록 합니다. 예를 들어, 제가 PR을 하나 올렸는데 미처 수정하지 못한 오타가 있어서 이를 즉시 수정하는 커밋을 바로 올렸다고 해 봅시다. 이 때, CI는 2번 돌아갑니다:

  • PR 생성 후 돌아가던 거 하나 (과거 커밋)
  • 그리고 오타 수정 커밋으로 돌아가는 하나 (최신 커밋)

이 설정은 같은 브랜치에 대해 최신 커밋에서 돌아가는 CI만 유지합니다. 즉, 과거 코드에 대한 CI는 새 커밋이 올라오면 즉시 중단하고, 새 커밋에 대해 새로 CI를 실행한다는 거죠.

CI 전용 local.properties를 GitHub 비밀에 추가

jobs:
  build_and_test:
    name: Build and test
    runs-on: ubuntu-latest

    steps:
      # ...

      # Prepare local properties
      - name: Prepare local properties
        run: printf '%s' "${{ secrets.LOCAL_PROPERTIES_CI }}" > local.properties

      # ...

CI에서만 사용할 local.propertiesGitHub 비밀에 추가했습니다.

일단, CI는 단순히 코드의 품질과 빌드 성공 여부만 검증하는 게 목적이기 때문에, 실제 실행을 전제하지 않습니다. 그래서 불필요한 값을 local.properties에 집어넣을 이유가 없고요. 그리고 prod용 local.properties를 CI 환경에서 그대로 사용할 경우, API 키나 API URL 등 중요한 정보가 노출될 위험이 있습니다. 따라서 보안 유지를 위해 CI용 local.properties를 별도로 관리합니다.

이 사항은 Confluence의 '환경 변수와 비밀' 문서에도 갱신되었음을 알려요.

🏞️ 스크린샷 (선택)

없음

🗣️ 리뷰 요구사항 (선택)

이 페이지 하단에 보시면 CI 돌아간 거 확인해볼 수 있어요. 한 번 살짝 들어가서 검토해보세요!

@i-meant-to-be i-meant-to-be requested a review from lyh5427 June 25, 2026 12:17
@i-meant-to-be i-meant-to-be self-assigned this Jun 25, 2026
@i-meant-to-be i-meant-to-be added the chore 파일 경로 변경, 의존성 등 비 개발 작업 label Jun 25, 2026
datulyh

This comment was marked as resolved.

@lyh5427 lyh5427 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

계정을 다른거 쓰고 있었네.. repo 권한 없다고 댓글 달았었는데 권한 있습니다 ㅎㅎ

@i-meant-to-be i-meant-to-be merged commit d1e8b2d into develop Jun 26, 2026
1 check passed
@i-meant-to-be i-meant-to-be deleted the chore/#5 branch June 26, 2026 00:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

chore 파일 경로 변경, 의존성 등 비 개발 작업

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[CHORE] CI 도입

3 participants