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

chore(*): 프로젝트 아키텍처 변경 #30

Merged
merged 4 commits into from
Jul 1, 2024
Merged

Conversation

Doeunnkimm
Copy link
Member

🎉 변경 사항

  • packages
    • core
      • sds
      • theme
      • config (typescript + eslint 통합)
    • web-domains
      • first-domains
      • second-domains

논의된 사항 반영했습니다 🧞‍♂️

🔗 링크

https://www.notion.so/depromeet/2a73917973da423db104d52bd54721b3?pvs=4

🙏 여기는 꼭 봐주세요!

Copy link
Collaborator

@semnil5202 semnil5202 left a 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'],
Copy link
Collaborator

Choose a reason for hiding this comment

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

린트 수정 요 PR 머지되면 작업해야겠군여 홍홍

@Doeunnkimm
Copy link
Member Author

Doeunnkimm commented Jun 30, 2024

@semnil5202

컴포넌트들이랑 토큰값들이 분리되어 있으면 하는 바람이 가장 큰데요.

보통 DS 자체가 모노레포이고, 그 하위 패키지 중 하나로 colors로 하여 토큰값을 관리하는 경우가 많은 것 것 같아요.

저희 같은 경우에는 sds 하위 src 하위에 패키지 분리없이 컴포넌트를 쌓아 넣을 것 같아 컴포넌트와 토큰값들을 분리하기를 원했어요!
(사실 말이 sds이지 공용 컴포넌트 폴더 성향이 강한 것 같아서 분리를 원했던 것...)

@Doeunnkimm
Copy link
Member Author

@Andrevile

8d115f8 sds 하위에 componentstheme 폴더로 구분하는 구조 반영하여, theme 패키지 제거했습니다~

Copy link
Collaborator

@semnil5202 semnil5202 left a comment

Choose a reason for hiding this comment

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

굳 확인했습니다!! 👍

@Doeunnkimm Doeunnkimm merged commit 9ae77e3 into main Jul 1, 2024
2 checks passed
@Doeunnkimm Doeunnkimm deleted the chore/architecture branch July 1, 2024 13:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants