-
Notifications
You must be signed in to change notification settings - Fork 0
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
chore(*): 프로젝트 아키텍처 변경 #30
Conversation
2b62e23
to
fdbae3c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
오호 제가 보기엔 깔끔하게 잘 변경된 것 같습니다. 저번에도 잠깐 의논했던 것 같은데 theme와 sds를 나눔으로써 얻는 이점? 이유가 있을까여?
개인적인 생각으론 theme가 sds에 종속되어 있다고 생각해서요. 근데 다시 생각해보면 모든 컴포넌트가 sds에 포함되지 않기도 하고, 해당 컴포넌트들도 theme를 사용할 수 있으므로 분리하는 것도 좋아 보이는 것 같습니다.
@@ -1,7 +1,7 @@ | |||
/** @type {import("eslint").Linter.Config} */ | |||
module.exports = { | |||
root: true, | |||
extends: ['@sambad/eslint-config/next.js'], | |||
extends: ['@sambad/config/eslint/next.js'], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
린트 수정 요 PR 머지되면 작업해야겠군여 홍홍
컴포넌트들이랑 토큰값들이 분리되어 있으면 하는 바람이 가장 큰데요. 보통 DS 자체가 모노레포이고, 그 하위 패키지 중 하나로 colors로 하여 토큰값을 관리하는 경우가 많은 것 것 같아요. 저희 같은 경우에는 sds 하위 src 하위에 패키지 분리없이 컴포넌트를 쌓아 넣을 것 같아 컴포넌트와 토큰값들을 분리하기를 원했어요! |
fdbae3c
to
8d115f8
Compare
8d115f8 sds 하위에 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
굳 확인했습니다!! 👍
🎉 변경 사항
논의된 사항 반영했습니다 🧞♂️
🔗 링크
https://www.notion.so/depromeet/2a73917973da423db104d52bd54721b3?pvs=4
🙏 여기는 꼭 봐주세요!