Conversation
|
There was a problem hiding this comment.
Pull request overview
This PR removes the TypeScript validation hook that was previously executed when Claude stops editing files. The hook is being removed because Claude now has built-in LSP (Language Server Protocol) support, making the external validation check redundant.
Changes:
- Removed the "Stop" hook configuration from
.claude/settings.json - Deleted the validation script
.claude/hooks/validation-check.shthat performed TypeScript type checking and linting
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
.claude/settings.json |
Removed the "Stop" hook configuration that triggered the validation check |
.claude/hooks/validation-check.sh |
Deleted the shell script that ran TypeScript type checking and ESLint validation |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #6313 +/- ##
=======================================
Coverage 42.59% 42.59%
=======================================
Files 2497 2497
Lines 43384 43384
Branches 9851 9851
=======================================
Hits 18479 18479
Misses 24868 24868
Partials 37 37 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Removed hook that checks TypeScript errors on Claude Code stop editing files
This is not needed anymore, since they added LSP support.