Skip to content

Upgrade pnpm11#12

Open
aaron1857 wants to merge 5 commits into
stagefrom
upgrade-pnpm11
Open

Upgrade pnpm11#12
aaron1857 wants to merge 5 commits into
stagefrom
upgrade-pnpm11

Conversation

@aaron1857
Copy link
Copy Markdown
Contributor

In this PR I brought the Docker caching I was doing in McKinney to this project. Additionally, I have made it compatible with pnpm 11. As a result, I did the following:

  1. Add dockerignore; less junk in the container means less junk that can change to invalidate the docker cache
  2. Remove .npmrc; apparently pnpm 11 doesn't care about this and wants settings moved to the pnpm workspace
  3. Add workdir since Tailwind wants to scan the entire current directory; this is an issue in the root directory
  • I don't think the Nuxt template uses the affected version of Tailwind. But I ran into this issue in McKinney, so I thought I'd do it here (it's best practice anyway).
  1. Switch to the LTS Alpine image since we probably don't want to be on the latest one anyway.
  2. pnpm kept trying to install node_modules again, so I had to do some package moving around BS to only get Prisma into the deployment container
  • This has the effect of reducing the image size since the node modules are smaller now.
  • What I did was put prisma in the dev dependencies so I can cleanly install it by telling prisma to only install the dev dependencies, though I put the no scripts flag since it kept trying to run the nuxt scripts (and we don't have nuxt here)
  • Then I manually ran all the scripts of the dev dependencies we need to get prisma to work.

@aaron1857 aaron1857 requested a review from Vikachubro21 May 10, 2026 20:35
@Vikachubro21
Copy link
Copy Markdown
Contributor

Removed shameful hoisting to help pnpm performance, re-added tailwind to the list of hoisted dependencies to ensure compatibility. Updated CI/CD to push ONLY prod/stage tags and not $IMAGE_TAG.

Copy link
Copy Markdown
Contributor

@Vikachubro21 Vikachubro21 left a comment

Choose a reason for hiding this comment

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

LGTM

@Vikachubro21 Vikachubro21 requested a review from TusharW4ni May 11, 2026 23:15
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.

2 participants