Skip to content

chore: remvoe husky - #432

Merged
imantsk merged 6 commits into
core-betafrom
husky/core
Aug 5, 2026
Merged

chore: remvoe husky#432
imantsk merged 6 commits into
core-betafrom
husky/core

Conversation

@ramiy

@ramiy ramiy commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

Removing redundant dev dependency. husky is a wrapper. We can replace this dependency.

@sheabunge

Copy link
Copy Markdown
Member

Good change in my opinion; I'll leave it for @imantsk to merge as he is the one who introduced Husky, I believe.

@ramiy
ramiy requested a review from imantsk August 4, 2026 09:43
@code-snippets-bot

Copy link
Copy Markdown
Contributor
  • Medium — package.json:11: Installing dependencies configures Git to use .githooks, but the replacement hook is at .github/pre-commit and is not executable, so pre-commit linting is silently disabled. Put the hook at .githooks/pre-commit with mode 100755, or configure .github and mark the file executable.

  • Low — CONTRIBUTING.md:74: Contributor setup still says Husky is installed and activated even though this PR removes it, leaving the documented hook workflow inaccurate. Update the Husky references and setup steps to match the replacement hook mechanism.

@ramiy

ramiy commented Aug 4, 2026

Copy link
Copy Markdown
Contributor Author
  • Medium — package.json:11: Installing dependencies configures Git to use .githooks, but the replacement hook is at .github/pre-commit and is not executable, so pre-commit linting is silently disabled. Put the hook at .githooks/pre-commit with mode 100755, or configure .github and mark the file executable.
  • Low — CONTRIBUTING.md:74: Contributor setup still says Husky is installed and activated even though this PR removes it, leaving the documented hook workflow inaccurate. Update the Husky references and setup steps to match the replacement hook mechanism.

Fixed.

@imantsk
imantsk merged commit 21ec231 into core-beta Aug 5, 2026
9 checks passed
@imantsk
imantsk deleted the husky/core branch August 5, 2026 08:23
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.

4 participants