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

[FEAT] #33 김성용 #39

Merged
merged 1 commit into from
Dec 20, 2024
Merged

[FEAT] #33 김성용 #39

merged 1 commit into from
Dec 20, 2024

Conversation

yongseong2
Copy link
Contributor

@yongseong2 yongseong2 commented Dec 6, 2024

Turborepo-도입기

늦게 올려 죄송합니다!!

@yongseong2 yongseong2 self-assigned this Dec 6, 2024
@tiaz0128
Copy link
Contributor

tiaz0128 commented Dec 7, 2024

@yongseong2
성용님 글 잘읽었습니다!

깔금하게 필요한 개념과 자신의 경험을 적절히 잘 녹여놓은 글인것 같습니다.

조금 팁을 드리자면

도입기나 개선한 글은 객관적인 수치화된 데이터가 있으면 더 좋은 글이 되는것 같습니다.

그래서 마무리에 적힌 '빌드 속도' 같은 측정 가능한 것들은 얼마나 개선되었는지 비교 자료가 있으면 반영하시면 더 좋을 듯 합니다!

CI/CD에서는 각 구조는 어떠했는지도 추가해봐도 좋을듯 합니다!

@yongseong2
Copy link
Contributor Author

@yongseong2 성용님 글 잘읽었습니다!

깔금하게 필요한 개념과 자신의 경험을 적절히 잘 녹여놓은 글인것 같습니다.

조금 팁을 드리자면

도입기나 개선한 글은 객관적인 수치화된 데이터가 있으면 더 좋은 글이 되는것 같습니다.

그래서 마무리에 적힌 '빌드 속도' 같은 측정 가능한 것들은 얼마나 개선되었는지 비교 자료가 있으면 반영하시면 더 좋을 듯 합니다!

CI/CD에서는 각 구조는 어떠했는지도 추가해봐도 좋을듯 합니다!

수치화된 데이터가 있으면 더 신뢰성 있는 글이 있을거 같네요!! CI/CD도 도입하면서 트러블슈팅 많이 했었는데 관련 내용도 녹여봐겠습니다ㅎㅎ 감사합니다!

@munyeol-Yoon
Copy link
Contributor

성용님 안녕하세요!
터보레포에 대한 글 잘 읽었습니다.
글에 경험이 잘 묻어나 인상깊었습니다...!

성용님의 글을 보고 저또한 이런 경험을 해보고 싶다는 생각이 들었습니다!
감사합니다. 고생하셨습니닷!

@S2uJeong
Copy link
Contributor

안녕하세요 성용님! 해당 글에 대해 리뷰를 남기지 않았던 것을 뒤늦게 알았습니다.ㅠㅠ

좋았던 점

  • 그림 자료가 이쁘다
  • 새로운 개념에 대해 장점부터 적용방법 적용 후 결과까지 상세히 적어 전달되었다.

제안점

  • 오타 : 간하다 -> 간단하다
    image
  • 시간이 단축된 정도를 수치적으로 나타낼 수 있다면 이해가 더 쉬울 것 같아요!
  • 어떤 기준으로 모놀리틱을 멀티 모듈로 나눴는지 여쭤봐도 될까요?

@yongseong2
Copy link
Contributor Author

@S2uJeong
안녕하세요 수정님
오타 찾아주셔서 감사합니다!
제일 주된 이유는

  1. 번들사이즈 크기
  2. 컴포넌트 공유

이 두가지 이유로 멀티모듈로 나누게 되었습니다. 번들사이즈를 줄이면서 컴포넌트가 공유되는 시스템을 지원하게끔 하고 싶어 모노레포로 변경하였습니다.

@yongseong2 yongseong2 merged commit 8515d40 into main Dec 20, 2024
@yongseong2 yongseong2 deleted the 10/yongseong2 branch December 20, 2024 23:31
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