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

init design system #9

Merged
merged 19 commits into from
Jan 22, 2024
Merged

init design system #9

merged 19 commits into from
Jan 22, 2024

Conversation

murjune
Copy link
Member

@murjune murjune commented Jan 22, 2024

관련 이슈

  • closed #이슈넘버

작업한 내용

  • : Theme 설정
  • : Color 설정
  • : Type(Typography) 설정

PR 포인트

@murjune murjune self-assigned this Jan 22, 2024
@murjune murjune requested a review from ham2174 January 22, 2024 12:22
Copy link
Collaborator

@ham2174 ham2174 left a comment

Choose a reason for hiding this comment

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

리뷰 남겨유~! @murjune

@@ -0,0 +1,10 @@
@Suppress("DSL_SCOPE_VIOLATION") // TODO: Remove once KTIJ-19369 is fixed
plugins {
alias(libs.plugins.funch.feature)
Copy link
Collaborator

Choose a reason for hiding this comment

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

alias(libs.plugins.funch.feature) 요거 플러그인은 피처 모듈에서만 사용되는거 아닌가요..?
build-logic에서 CommonConfigs.kt 파일에서 KotlinSerializationPlugin, AndroidHiltPlugin를 사용하고 있는데 디자인 시스템에서 모듈에서 사용이 되는지 궁금합니다!!

plugins {
    id("com.android.library")
    alias(libs.plugins.funch.kotlin)
    alias(libs.plugins.funch.compose)
}

요렇게 작성하는건 어떤가용??

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,10 @@
@Suppress("DSL_SCOPE_VIOLATION") // TODO: Remove once KTIJ-19369 is fixed
Copy link
Collaborator

@ham2174 ham2174 Jan 22, 2024

Choose a reason for hiding this comment

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

@Suppress("DSL_SCOPE_VIOLATION") 요구문은 찾아보니까 그래들 8.2이상이면 필요 없다고 합니다~! 링크
image

Copy link
Member Author

Choose a reason for hiding this comment

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

오 이거 빼는걸 깜빡했네유 반영하겠습니다!

@murjune murjune merged commit 12aafc7 into develop Jan 22, 2024
2 checks passed
@murjune murjune deleted the feature/init-design-system branch January 22, 2024 13:48
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