Remove husky-based pre-commit formatting#42582
Conversation
There was a problem hiding this comment.
Claude Code Review
This repository is configured for manual code reviews. Comment @claude review to trigger a review and subscribe this PR to future pushes, or @claude review once for a one-time review.
Tip: disable this comment in your organization's Code Review settings.
There was a problem hiding this comment.
Pull request overview
This PR removes the Husky/lint-staged-based pre-commit formatting setup that was introduced previously, reverting the repo back to not installing/overwriting local git hooks via prepare.
Changes:
- Removed
huskyandlint-stagedfrompackage.json(including thepreparescript andlint-stagedconfig). - Deleted the
.husky/pre-commithook that ranyarn lint-staged. - Updated
yarn.lockto remove the Husky entry (but it still appears to retainlint-staged).
Reviewed changes
Copilot reviewed 2 out of 3 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
package.json |
Removes Husky/lint-staged tooling and associated config/scripts. |
yarn.lock |
Removes Husky lock entry (needs follow-up cleanup for lint-staged). |
frontend/pages/hosts/ManageHostsPage/ManageHostsPage.tsx |
Contains unrelated new constants likely added accidentally (currently problematic). |
.husky/pre-commit |
Removes the pre-commit hook that ran formatting. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #42582 +/- ##
========================================
Coverage 66.64% 66.65%
========================================
Files 2532 2533 +1
Lines 202930 203042 +112
Branches 9026 9078 +52
========================================
+ Hits 135243 135332 +89
- Misses 55440 55460 +20
- Partials 12247 12250 +3
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Undoes #42470, which overwrites local git hook configuration