Remove pnpm support from build scripts

Created on 12 September 2024, 3 months ago

Problem/Motivation

We are not using pnpm for a long time now, lock files are removed, etc. We should also remove pnpm from build step in package.json:

"build-install": "((type pnpm && pnpm install) || npm install)",
📌 Task
Status

Fixed

Version

1.0

Component

BS Base

Created by

🇷🇸Serbia pivica

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Merge Requests

Comments & Activities

  • Issue created by @pivica
  • 🇷🇸Serbia pivica

    Actually just figured we removed it from our custom child themes that are using bs_bootstrap theme. However I just noticed we still have pnpm-lock.yaml in bs_base and it is making problems if you used `npm install` and then `npm run build` which will try to use pnpm first if available and throw all kind of errors because of previous `npm install`.

    Also npm feels much faster now and there are no benefits using pnpm in ddev enviroment, so let's remove pnpm support now and not latter. You can still use it in your custom themes if needed, just edit your child themes package.json and bring back pnpm there in build scripts.

  • Pipeline finished with Skipped
    3 months ago
    #281553
    • pivica committed 328254f0 on 8.x-1.x
      Issue #3473974 by pivica: Remove pnpm support from build scripts
      
  • Status changed to Fixed 3 months ago
  • 🇷🇸Serbia pivica

    Merged.

  • Automatically closed - issue fixed for 2 weeks with no activity.

Production build 0.71.5 2024