Skip to content

Create Week10 Mission1#49

Open
Hyerin-Jeong wants to merge 1 commit into
mainfrom
Week10/LinLin
Open

Create Week10 Mission1#49
Hyerin-Jeong wants to merge 1 commit into
mainfrom
Week10/LinLin

Conversation

@Hyerin-Jeong
Copy link
Copy Markdown
Contributor

📝 미션 번호

Week10 Mission1

📋 구현 사항

  1. 배포용 AAB 빌드
  • [o] 빌드된 AAB 파일 경로 PR에 업로드하기
    경로: app/build/outputs/bundle/release/app-release.aab
  1. Claude Code로 코드 리뷰 받기
  • [o]Claude Code CLI 설치하고 프로젝트 루트에서 실행하기

  • [o] 본인이 작성한 코드 선택해서 코드 리뷰 요청하기

  • [o]리뷰 결과를 바탕으로 실제로 코드 개선해보기

📎 스크린샷

✅ 체크리스트

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

🤔 질문 사항

Co-authored-by: Cursor <cursoragent@cursor.com>
@Dawon-Y Dawon-Y self-requested a review June 1, 2026 08:06
@Dawon-Y
Copy link
Copy Markdown
Contributor

Dawon-Y commented Jun 1, 2026

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


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

빌드 명령

Windows 기준(Week10/LinLin 루트에서):

.\gradlew.bat bundleRelease

AAB 생성 경로(과제에서 “PR에 기재” 요구)

RELEASE.md에 경로가 적혀있습니다:

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

PR 본문에 그대로 붙여넣으면 됩니다.

서명/키 관련 (현재 구성 체크)

  • app/build.gradle.kts에서 keystore.properties를 읽어 signingConfigs.release를 설정하도록 되어 있음
  • keystore.properties.example, local.properties.example를 제공한 것도 Good
  • 단, signingConfigs.create("release")에서 storeFilePath 없으면 return@create로 빠지는데,
    • 그 상태에서 release { signingConfig = signingConfigs.getByName("release") }로 묶으면 로컬 환경에 따라 릴리즈 빌드가 실패할 수 있어요.
    • 과제 제출용으로는 “keystore.properties를 반드시 채워야 한다”가 전제라 괜찮지만, README/RELEASE.md에 이 전제를 더 강하게 써주면 실수 줄어듭니다.

2) Claude Code(또는 CLI)로 코드 리뷰 받고 개선 반영

현재 상태: 문서화가 이미 잘 되어 있음

  • CLAUDE.md: 프로젝트 개요, 아키텍처, 기술스택, 빌드 방법, AI 협업 원칙까지 정리
  • RELEASE.md: “코드 리뷰 반영 사항(Week10)” 표로 개선사항을 요약

3) 추가로 칭찬/주의 포인트

좋은 점

  • BuildConfigField("REQRES_API_KEY", ...)로 키를 local.properties에서 주입하게 만든 점
    → 과제에서 자주 나오는 “키 커밋” 사고를 방지합니다.
  • 문서(CLAUDE.md, RELEASE.md)가 채점자가 원하는 정보(AAB 경로/개선 요약)를 바로 찾을 수 있게 되어 있음

주의 포인트(선택 개선)

  • compileSdk { version = release(36) { minorApiLevel = 1 } } / targetSdk = 36은 환경에 따라 빌드가 막힐 수 있습니다.
    • 과제 환경이 최신이라면 OK지만, 팀/채점자 환경이 34/35면 실패할 수 있습니다.
    • 제출 안정성이 목표면 compileSdk = 35 또는 “빌드 가능한 SDK 버전”으로 맞추는 게 안전합니다.

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

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