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

fix: remove npm version requirement #1047

merged 1 commit into from Nov 14, 2021

fix: remove npm version requirement #1047

merged 1 commit into from Nov 14, 2021


Copy link

@iiroj iiroj commented Nov 14, 2021

This fixes issue #1046

@iiroj iiroj requested a review from okonet November 14, 2021 09:49
Copy link
Collaborator Author

iiroj commented Nov 14, 2021

Ping @okonet let's remove the engines.npm field because it's not really relevant to installing lint-staged in a project, only development.

Copy link

codecov bot commented Nov 14, 2021

Codecov Report

Merging #1047 (41bc9df) into master (9f9213d) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##            master     #1047   +/-   ##
  Coverage   100.00%   100.00%           
  Files           20        20           
  Lines          577       577           
  Branches       156       156           
  Hits           577       577           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9f9213d...41bc9df. Read the comment docs.

@okonet okonet merged commit e50d6d4 into master Nov 14, 2021
20 checks passed
@iiroj iiroj deleted the remove-npm-version branch November 14, 2021 10:41
Copy link

🎉 This PR is included in version 12.0.2 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
None yet

Successfully merging this pull request may close these issues.

Node 16.13 LTS is not allowed
2 participants