Skip to content

Conversation

salmanmkc
Copy link
Contributor

  • Replace husky install with husky in prepare script
  • Remove shebang and husky.sh import from pre-commit hook
  • Maintain existing

V9 is a non-breaking change is is backwards compatible so we don't need to upgrade this in any rush even though v9 is now in main.

- Replace `husky install` with `husky` in prepare script
- Remove shebang and husky.sh import from pre-commit hook
- Maintain existing
@Copilot Copilot AI review requested due to automatic review settings August 30, 2025 03:46
@salmanmkc salmanmkc requested a review from a team as a code owner August 30, 2025 03:46
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR migrates Husky configuration from v8 to v9 format to modernize the pre-commit hook setup. The changes simplify the configuration while maintaining backward compatibility.

Key changes:

  • Updated the prepare script to use the new husky command instead of husky install
  • Simplified the pre-commit hook by removing the shebang and husky.sh import boilerplate
  • Consolidated the pre-commit commands into a single line

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
src/Misc/expressionFunc/hashFiles/package.json Updated prepare script to use new Husky v9 command format
.husky/pre-commit Simplified hook format by removing boilerplate and consolidating commands

@salmanmkc salmanmkc enabled auto-merge (squash) September 1, 2025 09:12
@salmanmkc salmanmkc merged commit f8ae5bb into main Sep 1, 2025
9 checks passed
@salmanmkc salmanmkc deleted the salmanmkc/husky-v9-update branch September 1, 2025 09:16
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.

3 participants