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.

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

[CT-1-2] hilt 추가, ktlint 추가, buildSrc 작성 #10

Merged

Conversation

nosorae
Copy link
Collaborator

@nosorae nosorae commented May 11, 2024

close #2

CT-1-1 PR을 먼저 보시는 게 흐름상 맞습니다.

  • hilt 세팅 및 빌드 확인했습니다.
  • ktlint 세팅 및 정상작동 확인했습니다.
  • buildSrc 적용하였고 불필요한 의존성은 제거했습니다. (후에 더 발견된다면 제거 또는 필요하다면 추가 예정입니다.)

nosorae added 16 commits May 11, 2024 16:27
…2-dependencies

# Conflicts:
#	app/.gitignore
#	app/build.gradle.kts
#	app/src/androidTest/java/com/yessorae/chartrainer/ExampleInstrumentedTest.kt
#	app/src/main/AndroidManifest.xml
#	app/src/test/java/com/yessorae/chartrainer/ExampleUnitTest.kt
#	build.gradle.kts
#	data/build.gradle.kts
#	data/src/androidTest/java/com/yessorae/data/ExampleInstrumentedTest.kt
#	data/src/test/java/com/yessorae/data/ExampleUnitTest.kt
#	domain/build.gradle.kts
#	presentation/build.gradle.kts
#	presentation/src/androidTest/java/com/yessorae/presentation/ExampleInstrumentedTest.kt
#	presentation/src/test/java/com/yessorae/presentation/ExampleUnitTest.kt
@nosorae nosorae requested a review from f-lab-nathan May 11, 2024 07:47
@nosorae nosorae self-assigned this May 11, 2024
@nosorae nosorae added feature settings This issue or pull request already exists and removed feature labels May 11, 2024
Copy link
Collaborator

@f-lab-nathan f-lab-nathan left a comment

Choose a reason for hiding this comment

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

커밋 잘게 나누는 것 좋은 습관입니다. 👍

nit; Presentation과 UI를 분리하는 것도 고려해볼 수 있습니다.

)
}
}
compileOptions {
sourceCompatibility = JavaVersion.VERSION_1_8
targetCompatibility = JavaVersion.VERSION_1_8
sourceCompatibility = JavaVersion.VERSION_17
Copy link
Collaborator

Choose a reason for hiding this comment

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

Q; 8에서 17로 변경할 때 어떤 이점이 있을까요?

Copy link
Collaborator Author

@nosorae nosorae May 11, 2024

Choose a reason for hiding this comment

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

명확한 이점을 찾아 올린 것은 아닙니다.
compileDebugJavaWithJavac와 kaptGenerateStubsDebugKotlin 빌드에러를 해결하기 위해 수정했습니다.

에러내용:
Caused by: org.gradle.api.GradleException: 'compileDebugJavaWithJavac' task (current target is 1.8) and 'kaptGenerateStubsDebugKotlin' task (current target is 17) jvm target compatibility should be set to the same Java version.

Base automatically changed from feature/ct-1-1-gitignore-and-module to feature/ct-1-initial-project-setup May 11, 2024 11:45
@f-lab-nathan f-lab-nathan merged commit 6ef5982 into feature/ct-1-initial-project-setup May 11, 2024
1 check passed
@nosorae nosorae deleted the feature/ct-1-2-dependencies branch May 11, 2024 13:08
@nosorae nosorae added this to the 최소 기능 구현 milestone May 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
settings This issue or pull request already exists
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants