-
Notifications
You must be signed in to change notification settings - Fork 8
fix ts check #507
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 ts check #507
Conversation
0925326
to
1ef2dba
Compare
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.
Pull Request Overview
This PR addresses TypeScript compilation issues by updating TypeScript configurations, fixing import paths, and correcting type definitions. The changes ensure the project builds correctly with proper type checking across all packages and applications.
Key changes include:
- Standardized TypeScript configuration with proper output directories and build settings
- Updated package paths and imports to resolve module resolution issues
- Fixed type definitions and import statements throughout the codebase
Reviewed Changes
Copilot reviewed 27 out of 29 changed files in this pull request and generated 3 comments.
Show a summary per file
File | Description |
---|---|
tsconfig.base.json | Removed empty line for cleaner formatting |
packages//tsconfig..json | Added outDir and noEmit configurations for proper build output |
package.json | Updated TypeScript check command to use project references |
apps/template-*/tsconfig.json | Added workspace package path mappings |
apps/server/* | Fixed Prisma import paths and type definitions |
apps/events/* | Corrected route parameter naming and import statements |
apps/connect/src/routes/authenticate.tsx | Removed unused variable assignment |
Files not reviewed (1)
- pnpm-lock.yaml: Language not supported
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
closes #507