Skip to content

Conversation

@Maffooch
Copy link
Contributor

Standardize package versioning in package.json and package-lock.json by removing caret symbols to ensure consistent installations across environments.

@Maffooch Maffooch requested a review from mtesauro as a code owner October 27, 2025 18:03
@github-actions github-actions bot added the docs label Oct 27, 2025
Copy link
Member

@valentijnscholten valentijnscholten left a comment

Choose a reason for hiding this comment

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

I think during my recent tests during the shai hulud incident the conclusion was that pinning should still work even with carets, especially if you use npm ci.
But it (removing carets) won't hurt I guess.

@valentijnscholten valentijnscholten added this to the 2.52.0 milestone Oct 27, 2025
Copy link
Contributor

@mtesauro mtesauro left a comment

Choose a reason for hiding this comment

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

Approved.

I'd guess this is to keep the docs site generating consistently while allowing us to choose when to update dependencies only needed to build the static site.

@mtesauro mtesauro merged commit 00d3fae into bugfix Oct 30, 2025
278 of 279 checks passed
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.

6 participants