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

chore: upgrade husky@8 and lint-staged@13 #206

Merged
merged 1 commit into from
Oct 14, 2022
Merged

chore: upgrade husky@8 and lint-staged@13 #206

merged 1 commit into from
Oct 14, 2022

Conversation

chimurai
Copy link
Contributor

Upgrade husky to v8 and lint-staged to v13

New Husky:

  • Zero dependencies and lightweight (6 kB)
  • Powered by modern new Git feature (core.hooksPath)
  • Follows npm and Yarn best practices regarding autoinstall

@@ -58,20 +58,14 @@
"build": "babel --out-dir build src --ignore src/__tests__ --source-maps --delete-dir-on-start --minified --no-comments",
"build:watch": "npm run build -- --watch",
"lint": "eslint ./src",
"prepare": "npm run lint && npm run test && npm run build",
"prepare": "husky install && npm run lint && npm run test && npm run build",
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Why husky doesn't autoinstall anymore

https://blog.typicode.com/husky-git-hooks-autoinstall/

Copy link
Collaborator

@tizmagik tizmagik left a comment

Choose a reason for hiding this comment

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

Thanks!

@tizmagik tizmagik merged commit 67cde3c into nytimes:main Oct 14, 2022
@chimurai chimurai deleted the husky-update branch October 14, 2022 09:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants