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

docs: Monorepo 정리 추가 #7

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

kanghyun98
Copy link
Contributor

모노레포 yarn-worspace 실습을 위해 공부하다가 모노레포 자체에 대한 간단한 정리도 진행했는데,
여기에 추가되면 좋을 것 같아서 올렸습니다..!

혹시 추가 및 수정하고 싶은 내용 있으시면 저한테 말씀해주셔도 좋고, 직접 커밋을 작성하셔도 좋습니다!!

Copy link
Member

@yoo-jimin127 yoo-jimin127 left a comment

Choose a reason for hiding this comment

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

아주 깔끔하고 필요한 부분들을 쏙쏙 집어주신 정리내용인 것 같습니다.
추후 패키지매니저 별 모노레포 구축 과정 md파일 및 실습 코드가 올라오면 링크로 연결되는 테이블 하나만 추가하면 될 것 같아요!!
정리 너무 수고하셨습니다~~!

Comment on lines +32 to +44
### 모노레포가 항상 옳을까?

아니다. 멀티레포와 모노레포의 장단점이 교차되기 때문에 적절한 상황에 맞게 사용해야 한다.

<br>

모노레포가 적절한 상황은 아래와 같다. ([Naver D2 글 참고](https://d2.naver.com/helloworld/0923884))

- 유사한 제품의 집합
- 여러 프로젝트의 변화를 한눈에 파악해야 할 때
- 호스트 애플리케이션을 플러그인 등으로 확장할 때
- 공통 기능을 재사용하는 관련된 프로젝트의 집합
- 유사한 DevOps로 구성된 프로젝트의 집합
Copy link
Member

Choose a reason for hiding this comment

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

저도 프로젝트에서 모노레포 도입을 고려하면서 공통적으로 사용할 컴포넌트가 뚜렷하지 않은 상황에서 모노레포 구축의 필요성에 대해 고민했었는데, 정리를 잘 해주셔서 더 명확한 답변을 내릴 수 있게 된 것 같습니다!
링크 걸어주신 글 너무 좋아요!

Copy link
Contributor

@ballsona ballsona left a comment

Choose a reason for hiding this comment

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

오 모노레포 관련 정리해주셔서 감사함다!
모노레포 등장 배경이랑 도입하기에 적절한 상황이 정리되어있어서 좋네요,, 👍👍
수고하셨습니다 :)


### 모노레포가 항상 옳을까?

아니다. 멀티레포와 모노레포의 장단점이 교차되기 때문에 적절한 상황에 맞게 사용해야 한다.
Copy link
Contributor

Choose a reason for hiding this comment

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

모노레포를 도입했을 경우의 단점도 들어가도 나쁘지 않을것 같아요!

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.

4 participants