Skip to content

Switch to pnpm (and update some dependencies)#2032

Merged
mcm001 merged 1 commit intoPhotonVision:mainfrom
Gold856:pnpm
Aug 4, 2025
Merged

Switch to pnpm (and update some dependencies)#2032
mcm001 merged 1 commit intoPhotonVision:mainfrom
Gold856:pnpm

Conversation

@Gold856
Copy link
Member

@Gold856 Gold856 commented Aug 4, 2025

Description

pnpm is much faster than npm for installing dependencies, and it prevents accidently depending on dependencies not listed in package.json. Some dependencies were also scoped to devDependencies to just to neaten things up. Caching has also been enabled in CI, so commits that are pushed in quick succession finish building faster.

Meta

Merge checklist:

  • Pull Request title is short, imperative summary of proposed changes
  • The description documents the what and why
  • If this PR changes behavior or adds a feature, user documentation is updated
  • If this PR touches photon-serde, all messages have been regenerated and hashes have not changed unexpectedly
  • If this PR touches configuration, this is backwards compatible with settings back to v2024.3.1
  • If this PR touches pipeline settings or anything related to data exchange, the frontend typing is updated
  • If this PR addresses a bug, a regression test for it is added

@Gold856 Gold856 requested a review from a team as a code owner August 4, 2025 05:47
@Gold856 Gold856 added frontend Having to do with PhotonClient and its related items website anything relating to https://photonvision.org labels Aug 4, 2025
Copy link
Contributor

@mcm001 mcm001 left a comment

Choose a reason for hiding this comment

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

Please also update building-photon.md if we are dropping npm entirely

@Gold856
Copy link
Member Author

Gold856 commented Aug 4, 2025

Already done.

@mcm001 mcm001 merged commit ad64bfe into PhotonVision:main Aug 4, 2025
68 of 69 checks passed
@Gold856 Gold856 deleted the pnpm branch August 5, 2025 01:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

frontend Having to do with PhotonClient and its related items website anything relating to https://photonvision.org

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants