-
Notifications
You must be signed in to change notification settings - Fork 104
Migrate eslint config #214
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
Conversation
|
📦 Build Artifacts Available |
|
📦 Build Artifacts Available |
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 overhauls the project’s ESLint setup to the new Flat ESLint format with TypeScript-aware and Next.js plugins, adjusts TypeScript and test configuration, and applies related import cleanups and minor code tweaks across the codebase.
- Migrated
eslint.config.jsto Flat ESLint with@typescript-eslint,@next/eslint-plugin-next, and Prettier integrations - Expanded
tsconfig.test.jsoninclude patterns and updated environment files and scripts to align with lint and test changes - Cleaned up import orders, removed duplicate lines, and refactored small utilities (e.g., removed unnecessary
async, cast event values)
Reviewed Changes
Copilot reviewed 34 out of 36 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| tsconfig.test.json | Broke out .ts and .tsx patterns, added test helper and mocks |
| eslint.config.js | Switched to Flat ESLint config, restructured plugins and rules |
| package.json | Updated lint script to auto-fix, adjusted dev dependencies |
| src/ui/lib/store/slices/benchmarks/benchmarks.api.ts | Changed dispatch signature to any, added ESLint disable comment |
| README.md | Removed detailed setup instructions; replaced with placeholder notice |
Comments suppressed due to low confidence (2)
README.md:162
- [nitpick] The README now only contains a placeholder and lacks essential setup and development instructions. Consider restoring or updating the usage guides to help contributors and users get started.
The UI is a WIP, check more recent PRs for the latest updates
tests/ui/unit/utils/interpolation.test.ts:39
- [nitpick] The naming of
expectedandactualhere is semantically reversed—expectedholds the interpolated value andactualholds the reference. Consider renaming them (e.g.,interpolatedandreference) to improve clarity.
const loopedValuesToTest: { expected: number; actual: number }[] = [];
Co-authored-by: Copilot <[email protected]>
|
📦 Build Artifacts Available |
|
📦 Build Artifacts Available |
Co-authored-by: Mark Kurtz <[email protected]> Co-authored-by: Copilot <[email protected]>
Co-authored-by: Mark Kurtz <[email protected]> Co-authored-by: Copilot <[email protected]> Signed-off-by: dalthecow <[email protected]>
Co-authored-by: Mark Kurtz <[email protected]> Co-authored-by: Copilot <[email protected]>
Co-authored-by: Mark Kurtz <[email protected]> Co-authored-by: Copilot <[email protected]> Signed-off-by: dalthecow <[email protected]>
Co-authored-by: Mark Kurtz <[email protected]> Co-authored-by: Copilot <[email protected]>
Co-authored-by: Mark Kurtz <[email protected]> Co-authored-by: Copilot <[email protected]> Signed-off-by: dalthecow <[email protected]>
Co-authored-by: Mark Kurtz <[email protected]> Co-authored-by: Copilot <[email protected]> Signed-off-by: dalthecow <[email protected]>
No description provided.