Skip to content

Create Week10 Mission 1#48

Open
umckee5696 wants to merge 2 commits into
mainfrom
Week10/Yido
Open

Create Week10 Mission 1#48
umckee5696 wants to merge 2 commits into
mainfrom
Week10/Yido

Conversation

@umckee5696
Copy link
Copy Markdown
Contributor

📌 미션 번호

10주차 실전 미션


📋 구현 사항

1. 배포용 AAB 빌드

  • 빌드된 AAB 파일 경로 PR에 업로드하기
    • 경로: app/build/outputs/bundle/release/app-release.aab

2. Claude Code로 코드 리뷰 받기

  • Claude Code CLI 설치하고 프로젝트 루트에서 실행하기
  • 본인이 작성한 코드 선택해서 코드 리뷰 요청하기
  • 리뷰 결과를 바탕으로 실제로 코드 개선해보기

📷 스크린샷


✅ 체크리스트

  • 배포용 AAB 빌드 완료 (./gradlew bundleRelease)
  • Claude Code CLI로 코드 리뷰 수행 완료
  • 리뷰 결과 바탕으로 코드 개선 완료
  • Merge 하려는 브랜치가 올바르게 설정되어 있나요?
  • 에뮬레이터 또는 실제 기기에서 정상 동작하나요?
  • 불필요한 주석 및 Log가 제거되었나요?

🔥 질문 사항

@umckee5696 umckee5696 changed the title Week10/yido Create Week10 Mission 1 May 31, 2026
@umckee5696 umckee5696 requested a review from Dawon-Y May 31, 2026 13:00
@umckee5696 umckee5696 self-assigned this May 31, 2026
@Dawon-Y
Copy link
Copy Markdown
Contributor

Dawon-Y commented Jun 1, 2026

안녕하세요! 10주차 과제 진행하시느라 고생 많으셨습니다💚


1) 배포용 AAB 빌드 + AAB 경로 PR 업로드

빌드 명령

프로젝트 루트 Week10/Yido에서:

./gradlew bundleRelease

(Windows면 gradlew.bat bundleRelease)

AAB 파일 경로 (PR에 그대로 기재)

README.md에 이미 명시돼 있고, 표준 경로도 맞습니다:

app/build/outputs/bundle/release/app-release.aab

2) Claude Code로 코드 리뷰 받고 개선 반영 (증빙/구성)

현재 상태: README가 “체크리스트 증빙 문서” 역할을 함

Week10/Yido/README.md에 다음이 이미 체크되어 있어요:

  • AAB 경로 PR 업로드
  • Claude Code CLI로 코드 리뷰 수행
  • 리뷰 기반 코드 개선 완료

즉, 과제 제출용으로는 README 자체가 결과 보고서가 됩니다.


3) 지금 설정에서 딱 3개만 짚으면

(1) API 키 주입 방식 좋음

app/build.gradle.kts에서 local.propertiesAPI_KEY를 읽어서 BuildConfig.API_KEY로 노출

val apiKey = localProperties.getProperty("API_KEY")
    ?: throw GradleException("local.properties에 API_KEY를 설정하세요.")
buildConfigField("String", "API_KEY", "\"$apiKey\"")
  • “키 하드코딩/커밋” 사고를 막는 정석 방식입니다.

(2) 릴리즈 서명 설정도 “과제용 현실적”

  • RELEASE_STORE_FILE이 있을 때만 signingConfig 적용하도록 분기해둔 건,
    서명 정보 없는 환경에서도 build.gradle sync가 터지는 걸 줄여줍니다.
  • 다만 실제 bundleRelease는 Play 업로드 용이면 서명이 사실상 필요하니까, README에 “릴리즈 서명 필수”를 한 줄 더 써도 좋습니다.

(3) compileSdk/targetSdk 36은 환경 의존 가능

LinLin처럼 Yido도 compileSdk/targetSdk = 36이라, 채점/CI 환경이 최신 SDK가 아니면 빌드가 막힐 수 있습니다.

  • 과제 환경이 36을 요구/보장하면 OK
  • 아니면 제출 안정성 위해 34/35로 맞추는 것이 안전(가이드에 따라 판단)

PR에 적을 “최소 문구” 템플릿

PR 본문에 아래만 있으면 충분합니다:

  • AAB 경로: app/build/outputs/bundle/release/app-release.aab
  • 빌드 명령: ./gradlew bundleRelease
  • Claude Code 리뷰/개선: Week10/Yido/README.md에 체크리스트 및 결과 정리

워크북 진행하시느라 수고하셨습니다! 궁금한 점 있으시면 언제든 말씀해 주세요!

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants