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

core모듈 재세팅 했슴당 #24

Merged
merged 8 commits into from
Jun 18, 2024

Conversation

ddyeon
Copy link
Member

@ddyeon ddyeon commented Jun 17, 2024

개요

이슈 링크 혹은 PR 내용 요약

core 모듈도 모듈로 다시 분리했어유~!

작업 내용

실제 작업 내용

스크린샷 2024-06-17 오후 11 45 56

시연 화면 (option)

실행 스크린샷 혹은 영상 첨부

기능A 구현 기능B 구현 ...
...

To Reviers

리뷰어들에게 전할 말

디자인시스템모듈에 component랑 theme 패키지 우선 만들었어요~!
component에 공통 컴포넌트들 쓰면 될 것 같아서 우선 예시파일로 하나 넣어뒀슴당

Close

close#

@ddyeon ddyeon self-assigned this Jun 17, 2024
@ddyeon ddyeon requested review from Ahn-seokjoo and removed request for Ahn-seokjoo June 17, 2024 14:47
@ddyeon ddyeon added the feature develop feature label Jun 17, 2024
@ddyeon
Copy link
Member Author

ddyeon commented Jun 17, 2024

lint통과 왜케 어려운거야!!!!!!!!!!!!!!!!!!
테스트파일은 와일드카드 없애도 반영이 안되나벼

@@ -0,0 +1 @@
/build
Copy link
Collaborator

Choose a reason for hiding this comment

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

넣을거면 다넣징

Copy link
Collaborator

Choose a reason for hiding this comment

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

Copy link
Member Author

Choose a reason for hiding this comment

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

넹~~

@@ -0,0 +1 @@
/build
Copy link
Collaborator

Choose a reason for hiding this comment

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

넣다넣

Comment on lines +3 to +4
import org.junit.Assert.assertEquals
import org.junit.Test
Copy link
Collaborator

Choose a reason for hiding this comment

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

잘 빠진듯 ?! * 안들어가넹

Copy link
Collaborator

@Ahn-seokjoo Ahn-seokjoo left a comment

Choose a reason for hiding this comment

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

넣다만 ignore만 챙겨줘요 ㅎ;
그래도 잘 통과하셨네용 굿!

Copy link
Collaborator

@fbghgus123 fbghgus123 left a comment

Choose a reason for hiding this comment

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

@ddyeon
👍👍👍👍
근데 코어 모듈을 쪼갠 이유는 먼가요 ??

}
}

fun CoroutineScope.doraLaunch(block: suspend CoroutineScope.() -> Unit) =
Copy link
Collaborator

Choose a reason for hiding this comment

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

도라런치 ~

Copy link
Collaborator

Choose a reason for hiding this comment

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

이름 귀엽지

Comment on lines +6 to +33
android {
namespace = "com.mashup.dorabangs.core.coroutine"
compileSdk = 34

defaultConfig {
minSdk = 24

testInstrumentationRunner = "androidx.test.runner.AndroidJUnitRunner"
consumerProguardFiles("consumer-rules.pro")
}

buildTypes {
release {
isMinifyEnabled = false
proguardFiles(
getDefaultProguardFile("proguard-android-optimize.txt"),
"proguard-rules.pro",
)
}
}
compileOptions {
sourceCompatibility = JavaVersion.VERSION_18
targetCompatibility = JavaVersion.VERSION_18
}
kotlinOptions {
jvmTarget = "18"
}
}
Copy link
Collaborator

Choose a reason for hiding this comment

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

이런거 묶어서 한 번에 관리해주는 그런게 있엇던 것 같은데 나중에 찾아서 넣어볼게 ~

Copy link
Member Author

Choose a reason for hiding this comment

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

마자 컨벤션 플러그인말하는건가??!! 해줘.

@ddyeon
Copy link
Member Author

ddyeon commented Jun 18, 2024

@ddyeon 👍👍👍👍 근데 코어 모듈을 쪼갠 이유는 먼가요 ??

오빠가 잠들었을 때 그렇게 하기로 했어.

기존에 보여줬던 증분모듈 레퍼런스 참고해보니 core모듈도 쪼개서 필요한 부분에다 가져다 쓰더라구!
우리는 지금 core에 coroutine이랑 designsystem밖에 없어서 모든 feature에서 쓰고 있긴한데,,
나중에 core에 많이 추가되면 core안에서도 나눠서 사용 가능할듯~

@fbghgus123 fbghgus123 merged commit 2b4c7d3 into mash-up-kr:develop Jun 18, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature develop feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants