Skip to content

Conversation

@addaleax
Copy link
Collaborator

This was done through manual package-lock surgery since, for some inexplicable reason, npm decided to have a separate js-yaml dependency under cli-repl even though the hoisted one at the top level (4.1.1) would have matched cli-repl's requirements.

This was done through manual package-lock surgery since, for some
inexplicable reason, npm decided to have a separate js-yaml dependency
under cli-repl even though the hoisted one at the top level (4.1.1)
would have matched cli-repl's requirements.
@addaleax addaleax requested a review from a team as a code owner November 27, 2025 11:12
Copilot AI review requested due to automatic review settings November 27, 2025 11:12
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the js-yaml dependency version in the cli-repl package from ^4.1.0 to ^4.1.1 to align with the hoisted version at the project root, eliminating an unnecessary duplicate dependency.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@addaleax addaleax added the no-title-validation Skips validation of PR titles (conventional commit adherence + JIRA ticket inclusion) label Nov 27, 2025
@addaleax addaleax merged commit e5a7893 into main Nov 27, 2025
147 of 151 checks passed
@addaleax addaleax deleted the js-yaml-cli-repl branch November 27, 2025 13:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-title-validation Skips validation of PR titles (conventional commit adherence + JIRA ticket inclusion)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants