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.

Sign up for GitHub

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

Jump to bottom

#2 : 안드로이드 프로젝트 셋업 #4

Merged
merged 21 commits into from
Jun 16, 2024
Merged

#2 : 안드로이드 프로젝트 셋업 #4

merged 21 commits into from
Jun 16, 2024

Conversation

jeongjaino
Copy link
Member

1. 📄 관련된 이슈 및 소개

#2 안드로이드 프로젝트 셋업

2. 🔥 변경된 점

  1. 버전 카탈로그가 추가되었습니다.
  • 이전에 정의한 컨벤션에 따라 지정하였습니다. 자세한 사항은 노션 참고 ~!
  1. 그레들 컨벤션 플러그인이 추가되었습니다.
  • Application, Feature, Library, Hilt, Firebase, Compose 정도로 구성하였습니다.
  1. 모듈 구조를 만들었습니다.
  • 일단 domain, data, designsystem, core, feature, app 큰 영역으로 구성하였고, 나머지 모듈은 개발하면서 생성하고자 따로 추가하지 않았습니다.

3. ✅ 필수 체크 사항

  • 모듈 및 여러 플러그인 작업이 많아서 파일 갯수는 많으나 실제 확인해야할 부분은 버전 카탈로그와 그레들 플러그인 중심으로 봐주시면 감사하겠습니다 !
  • 첫 피알인 만큼 추가적으로 문서를 작성하였습니다.

4. 📸 작업물 사진 공유(선택)

5. 💡알게된 혹은 궁금한 사항

[ Kotlin 2.0.0v 대응하기 POST

  • 2.0.0v로 되면서 컴포즈 컴파일러를 젯브레인에서 관리하게 되었습니다.
  • 그에 따라 compileOptions과 같은 기본 설정값이 변경된 사항이 있었습니다. (시간나면 읽어보시면 좋을듯!~)
image

@jeongjaino jeongjaino added 🍻제이콥🍻 24기 정진호 🔥리뷰 기다리는 중🔥 PR을 올리고 코드리뷰를 기다리고 있는 상태입니다. 🧹허드렛일🧹 CHORE, 종속성 추가, 기능과 무관한 코드 작성 labels Jun 12, 2024
@jeongjaino jeongjaino requested a review from flash159483 June 12, 2024 14:35
@jeongjaino jeongjaino self-assigned this Jun 12, 2024
@jeongjaino jeongjaino linked an issue Jun 13, 2024 that may be closed by this pull request
[CHORE] : 프로젝트 셋업 #2
Closed
3 tasks
flash159483
flash159483 reviewed Jun 13, 2024
View reviewed changes
.gitignore Outdated Show resolved Hide resolved
...d-logic/convention/src/main/kotlin/com/bff/wespot/plugin/AndroidApplicationFirebasePlugin.kt Outdated Show resolved Hide resolved
domain/build.gradle.kts Show resolved Hide resolved
core/model/build.gradle.kts Show resolved Hide resolved
feature/auth/build.gradle.kts Show resolved Hide resolved
@jeongjaino jeongjaino closed this Jun 15, 2024
@jeongjaino jeongjaino reopened this Jun 15, 2024
@jeongjaino jeongjaino added ✏️수정 요청✏️ 코드 리뷰 후 코드 수정 요청 and removed 🔥리뷰 기다리는 중🔥 PR을 올리고 코드리뷰를 기다리고 있는 상태입니다. labels Jun 15, 2024
flash159483
flash159483 reviewed Jun 15, 2024
View reviewed changes
Copy link
Contributor

@flash159483 flash159483 left a comment

Choose a reason for hiding this comment

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

정말로 수고가 많으시군요. 중간에 시간 잠시내서 리뷰 달아봤어요

domain/build.gradle.kts Show resolved Hide resolved
app/build.gradle.kts Outdated Show resolved Hide resolved
@jeongjaino jeongjaino added 🌟머지 해주세요🌟 코드 리뷰가 완료된 뒤 PR을 올린사람이 Merge를 하면 되는 단계입니다. and removed ✏️수정 요청✏️ 코드 리뷰 후 코드 수정 요청 labels Jun 16, 2024
jeongjaino
jeongjaino commented Jun 16, 2024
View reviewed changes
Copy link
Member Author

@jeongjaino jeongjaino left a comment

Choose a reason for hiding this comment

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

ACTIONS CHECK

@jeongjaino
Copy link
Member Author

hello

jeongjaino
jeongjaino commented Jun 16, 2024
View reviewed changes
Copy link
Member Author

@jeongjaino jeongjaino left a comment

Choose a reason for hiding this comment

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

COMMENT CHECK ACTIONS CHECK HELLO

jeongjaino
jeongjaino commented Jun 16, 2024
View reviewed changes
Copy link
Member Author

@jeongjaino jeongjaino left a comment

Choose a reason for hiding this comment

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

PR 리뷰 액션 테스트, Comment 상태 전송

jeongjaino
jeongjaino commented Jun 16, 2024
View reviewed changes
Copy link
Member Author

@jeongjaino jeongjaino left a comment

Choose a reason for hiding this comment

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

찐막 테스트 Github aCtIOnS

@jeongjaino jeongjaino merged commit 227ab7b into main Jun 16, 2024
6 checks passed
@jeongjaino jeongjaino deleted the chore/jaino/#2 branch June 16, 2024 08:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@flash159483 flash159483 flash159483 approved these changes

Assignees

@jeongjaino jeongjaino

Labels
🌟머지 해주세요🌟 코드 리뷰가 완료된 뒤 PR을 올린사람이 Merge를 하면 되는 단계입니다. 🍻제이콥🍻 24기 정진호 🧹허드렛일🧹 CHORE, 종속성 추가, 기능과 무관한 코드 작성
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[CHORE] : 프로젝트 셋업
2 participants
@jeongjaino @flash159483