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

PB-1436 Switch to using pnpm instead of npm #1241

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from

Conversation

schtibe
Copy link
Contributor

@schtibe schtibe commented Feb 13, 2025

PNPM

pnpm is a system that re-uses node_modules across multiple folders and projects. This results in way faster installs and less disk usage

Differences

The main difference is that one needs to run pnpm install instead of npm install, pnpm run instead of npm run etc.

Catalogs and Workspaces

The file pnpm-workspaces.yaml is where this workspaces is defined. For one, it defines the packages that are included in this mono-repo. This helps with certain command and especially with cross-linking the packages.

Secondly, there's a catalog definition. In there are the packages defined, that should be the same across all of the sub-packages. This helps preventing the problem that one package might update to a newer version while another one stays at the same, resulting in a conflict.

Test link

@schtibe schtibe force-pushed the feat-pb-1436-pnpm branch 13 times, most recently from 5c4c768 to 639670e Compare February 14, 2025 10:21
@schtibe schtibe changed the title Feat pb 1436 pnpm PB-1436 Switch to using pnpm instead of npm Feb 14, 2025
@schtibe schtibe force-pushed the feat-pb-1436-pnpm branch 2 times, most recently from 6566499 to a2e188f Compare February 14, 2025 12:27
@schtibe schtibe requested a review from pakb February 14, 2025 12:40
@schtibe schtibe force-pushed the feat-pb-1436-pnpm branch 2 times, most recently from ae4562e to 23650a7 Compare February 14, 2025 12:44
Copy link
Contributor

@pakb pakb left a comment

Choose a reason for hiding this comment

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

as would see our greatest Kazakh hero : very niiice!

@schtibe schtibe force-pushed the feat-pb-1436-pnpm branch 6 times, most recently from e2a94ac to f3ce942 Compare February 21, 2025 12:47
Copy link

cypress bot commented Feb 21, 2025

web-mapviewer    Run #4642

Run Properties:  status check passed Passed #4642  •  git commit aa1227d5de: Fix missing chai dependency
Project web-mapviewer
Branch Review feat-pb-1436-pnpm
Run status status check passed Passed #4642
Run duration 05m 11s
Commit git commit aa1227d5de: Fix missing chai dependency
Committer Stefan Heinemann
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 0
Tests that did not run due to a developer annotating a test with .skip  Pending 22
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 233
View all changes introduced in this branch ↗︎

Adding a workspace configuration with a catalog to be used from all our
packages. This will require to run `pnpm install` on the top level and
it will install all required packages in our modularized packages.

The catalog in the workspace configuration helps having the same version
across all our sub-packages.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants