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

Improve monorepo builds #457

Merged
merged 18 commits into from
Jan 18, 2024
Merged

Improve monorepo builds #457

merged 18 commits into from
Jan 18, 2024

Conversation

SnowSuno
Copy link
Member

@SnowSuno SnowSuno commented Jan 16, 2024

요약 *

기존의 모노레포 빌드 방식을 개선합니다.

  • 번들러 및 빌드 관련 패키지 dependency 업데이트
  • tsconfigmoduletargetNodeNext 또는 ESNext로 변경
  • path alias를 통한 monorepo 간 패키지 의존 workaround 해결
    • Redundant한 tsconfig.build.json 삭제
    • @biseo/interfacedev 동작 방식을 build watch 방식으로 변경
    • subpath export 되지 않은 경로를 참조할 수 있던 문제 해결됨
    • 각 package 내부 internal absolute import(@/*) 가능해짐
  • eslintimport/resolver 관련 문제 해결

이후 Task *

  • 패키지 내부 internal absolute import의 unique prefix 제거 (ex: @biseo/web/* -> @/*)

Copy link

netlify bot commented Jan 16, 2024

Deploy Preview for biseo-preview ready!

Name Link
🔨 Latest commit 40e4989
🔍 Latest deploy log https://app.netlify.com/sites/biseo-preview/deploys/65a789482add4200081c003a
😎 Deploy Preview https://deploy-preview-457--biseo-preview.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@SnowSuno SnowSuno self-assigned this Jan 16, 2024
@SnowSuno SnowSuno added the enhancement New feature or request label Jan 16, 2024
@@ -1,6 +1,6 @@
{
"extends": ["@commitlint/config-conventional"],
"rules": {
"header-max-length": [2, "always", 50]
"header-max-length": [2, "always", 72]
Copy link
Member Author

Choose a reason for hiding this comment

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

그냥 이참에 commitlint 최대자 제한을 72자로 늘렸어요

72자가 recommended length더라고요
https://github.com/conventional-changelog/commitlint/blob/master/docs/reference-rules.md

@@ -10,14 +10,15 @@
"web": "pnpm -F @biseo/web",
"api": "dotenv -- pnpm -F @biseo/api",
"interface": "pnpm -F @biseo/interface",
"dev": "dotenv -- pnpm -r --stream dev",
"dev": "dotenv -- pnpm -r --stream --parallel dev",
Copy link
Member Author

Choose a reason for hiding this comment

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

@biseo/interfacedev(build watch) script가 추가됨에 따라 이 명령이 끝나지 않으므로 --parallel 옵션을 주어 동시에 실행되도록 함

@SnowSuno SnowSuno marked this pull request as ready for review January 16, 2024 19:22
Copy link
Member

@withSang withSang left a comment

Choose a reason for hiding this comment

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

👍
고생하셨습니다!

@SnowSuno SnowSuno merged commit 579e460 into main Jan 18, 2024
5 checks passed
@SnowSuno SnowSuno deleted the feat/update-monorepo-build branch January 18, 2024 12:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants