-
Notifications
You must be signed in to change notification settings - Fork 113
chore: enable eslint rule for consistent type import/export MCP-130 #470
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
chore: enable eslint rule for consistent type import/export MCP-130 #470
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.
Pull Request Overview
This pull request enables the ESLint rule for consistent type import/export as part of an effort to introduce stricter linting rules around configuration injection. The change enforces the use of type imports for TypeScript types and consistent type exports throughout the codebase.
- Enables
@typescript-eslint/consistent-type-importsrule withprefer: "type-imports"option - Enables
@typescript-eslint/consistent-type-exportsrule to ensure consistent export patterns - Updates all import statements throughout the codebase to use
import typefor type-only imports
Reviewed Changes
Copilot reviewed 97 out of 97 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| eslint.config.js | Adds new ESLint rules for consistent type imports and exports |
| Multiple .ts files | Updates import statements to use import type for TypeScript types and interfaces |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
Pull Request Test Coverage Report for Build 17156392072Details
💛 - Coveralls |
Proposed changes
This is part of the effort for introducing an eslint rule to only use injected config in source code. This PR enables consistent type import/export rule on which the next rule will build upon.
Checklist