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.

Sign up for GitHub

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

Jump to bottom

Remove specified versions of dependencies. #356

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Remove specified versions of dependencies. #356

wants to merge 1 commit into from

Conversation

fbosio
Copy link

@fbosio fbosio commented Jun 11, 2024 •
edited
Loading

Summary
Let npm decide which version to use automatically by removing specific ranges from package.json.

According to the npm docs, this change is valid.
https://docs.npmjs.com/cli/v10/configuring-npm/package-json#dependencies

This is a fix for #349 (comment).

Test Plan

Already ran :Prettier in vim8 and nvim with a test.html open containing the below line.

<!DOCTYPE html><html><head><title>test</title></head><body>test</body></html>

Running :Prettier produced the expected results.

<!doctype html>
<html>
        <head>
                <title>test</title>
        </head>
        <body>
                test
        </body>
</html>

@fbosio
Remove specified versions of dependencies.
74f7c31 
According to the npm docs, this change is valid.
https://docs.npmjs.com/cli/v10/configuring-npm/package-json#dependencies

So let npm decide which version to use.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@fbosio