-
Notifications
You must be signed in to change notification settings - Fork 59
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
fix!: dep-check tsx and jsx #1661
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
self review
@@ -2,7 +2,8 @@ | |||
"extends": "./src/config/tsconfig.aegir.json", | |||
"compilerOptions": { | |||
"outDir": "dist", | |||
"emitDeclarationOnly": true | |||
"emitDeclarationOnly": true, | |||
"jsx": "preserve" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
aegir build fails without this with the new fixtures, but it shouldn't impact output of aegir since there are no src/**/*.jsx
or src/**/*.tsx
files
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM - just a small suggestion to improve the .tsx
test.
Also: this should go out as a major since things that were not being depchecked before will start to become depchecked which may cause breakage.
## [45.0.0](v44.1.4...v45.0.0) (2024-10-28) ### ⚠ BREAKING CHANGES * `.tsx` and `.jsx` files are now checked for missing/unused dependencies ### Bug Fixes * dep-check tsx and jsx ([#1661](#1661)) ([326e281](326e281))
🎉 This PR is included in version 45.0.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
Adds configuration to dependency-check command to check for unused dependencies properly in jsx & tsx files.
Also adds tests