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
merged 1 commit into from Nov 14, 2021
Merged

fix: remove npm version requirement #1047

merged 1 commit into from Nov 14, 2021

Conversation

iiroj
Copy link
Member

@iiroj iiroj commented Nov 14, 2021

This fixes issue #1046

@iiroj iiroj requested a review from okonet November 14, 2021 09:49
@iiroj
Copy link
Member 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.

@codecov
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
@iiroj iiroj deleted the remove-npm-version branch November 14, 2021 10:41
@github-actions
Copy link
Contributor

🎉 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
Labels
Development

Successfully merging this pull request may close these issues.

Node 16.13 LTS is not allowed
2 participants