-
Notifications
You must be signed in to change notification settings - Fork 2
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
[BREAKING] Change Monorepo tool to Turborepo #470
Conversation
test release workflow test release workflow test release workflow test release workflow test release workflow test release workflow test release workflow test release workflow test release workflow test release workflow test release workflow test release workflow test release workflow test release workflow
4a7ef4f
to
eca528e
Compare
This reverts commit c5127c4.
ดูจาก repo GitOps แล้วคิดว่าน่าจะต้องแก้วิธีใส่ version ใหม่ เดี๋ยว merge อันนี้ไปก่อน แล้วเดี๋ยว เปิด PR ไปยัง repo GitOps ตอน deploy เดี๋ยวมาอีกที |
@@ -23,7 +23,7 @@ FROM pnpm AS installer | |||
WORKDIR /app | |||
|
|||
COPY --from=builder /app/out/json/ . | |||
COPY --from=builder /app/out/pnpm-lock.yaml ./pnpm-lock.yaml | |||
COPY pnpm-lock.yaml ./pnpm-lock.yaml |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why are we no longer using pruned lock file?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
because sometimes turbo incorrectly prunes the file and remove some files we need (e.g. mongoose and shit) vercel/turborepo#2791
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i figure it's fine if we use the full lockfile, since we pruned the package json anyways?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Huh, it looks fine over here. No missing modules.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can you try using pruned pnpm-lock and build reg-scraper image?
Why did you create this PR
What I'm planning to do
project.json
andnx.json
pnpm-lock.yaml
package.json
for each app and package withname
,version
, and general script likedev
,build
,lint
pnpm-lock.yaml
, and install external package with specific version for each app and packages.Done
Demo
No demo yet
Checklist
Related links
Resource https://github.com/saenyakorn/turborepo-versioning-demo/
Notes
I'll continuously update the progress. And I'll ready for review when I'm fully sure that our apps can be built and run properly.