sparcs-kaist / taxi-front

KAIST Taxi Party Matching Web Service
https://taxi.kaist.ac.kr
MIT License
13 stars 2 forks source link

# 699 Migrate to Monorepo #700

Closed SnowSuno closed 8 months ago

SnowSuno commented 8 months ago

Summary

It closes #699

pnpm workspace 기반 Monorepo 구조를 사용하기 위해 기존의 web 소스코드를 @taxi/web 하위 모듈로 이동합니다.

Further Work

netlify[bot] commented 8 months ago

Deploy Preview for taxi-dev-preview ready!

Name Link
Latest commit
Latest deploy log https://app.netlify.com/sites/taxi-dev-preview/deploys/65a766b1d410ec6e7a67f055
Deploy Preview https://deploy-preview-700--taxi-dev-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 commented 8 months ago

@withSang 시간 날때 CI/CD나 netlify preview, linter 등 영향 받는 부분 트래킹하는거 한번 도와주실 수 있나요?

SnowSuno commented 8 months ago

@withSang @cokia CI/CD 쪽 한번 봐주시면 감사하겠습니다! netlify 빌드 커맨드도 제가 임의로 바꿔도 되는지 모르겠어서 일단 놔두었습니다

14KGun commented 8 months ago

@SnowSuno

Dockerfile 빌드할 때, pnpm build 를 하면 package/web빌드와 package/app 빌드가 모두 될 것 같은데 Dockerfilepackage/webpackage/app 따로 만드는 것은 어떤가욤??

SnowSuno commented 8 months ago

@SnowSuno

Dockerfile 빌드할 때, pnpm build 를 하면 package/web빌드와 package/app 빌드가 모두 될 것 같은데 Dockerfilepackage/webpackage/app 따로 만드는 것은 어떤가욤??

앱은 앱 특성상 사실 도커파일이 필요 없을 것 같고, 웹용 도커 파일에서는 pnpm -F @taxi/web build 명령어(pnpm web build로 alias 설정되어 있습니다)로 빌드하도록 수정해야 합니다.

도커파일은 일부러 따로 수정하지 않고 놔두었던 건데, 그냥 이 부분까지 이 PR에서 수정하는게 나을 듯 하네요! 도커파일까지 고쳐서 다시 리뷰 요청드릴게요!

SnowSuno commented 8 months ago

Dockerfile까지 업데이트해서 다시 PR 열었습니다. @14KGun

ybmin commented 8 months ago

LGTM 수고하셨습니당

SnowSuno commented 8 months ago

netlify build 설정도 업데이트해두었습니다

SnowSuno commented 8 months ago

679 의 todo도 일부 해결해버린(?) 것 같네요