Skip to content

Upgrade eslint #1115

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

Merged
merged 3 commits into from
Jul 19, 2025
Merged

Upgrade eslint #1115

merged 3 commits into from
Jul 19, 2025

Conversation

scholarsmate
Copy link
Member

@scholarsmate scholarsmate commented Jul 19, 2025

This is a major upgrade of eslint which uses a different file format for the configuration.

@scholarsmate scholarsmate requested a review from Copilot July 19, 2025 18:41
Copy link
Contributor

@Copilot Copilot AI left a 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 upgrades ESLint from version 8 to version 9 and migrates the configuration format from the legacy .eslintrc JSON format to the new flat config format introduced in ESLint 9.

  • Upgrades ESLint dependency from ^8.57.0 to ^9.30.1
  • Replaces .eslintrc JSON configuration with new eslint.config.js flat config format
  • Maintains existing Prettier integration and configuration settings

Reviewed Changes

Copilot reviewed 3 out of 4 changed files in this pull request and generated 1 comment.

File Description
package.json Updates ESLint dependency to version 9.30.1
packages/client/.eslintrc Removes legacy ESLint configuration file
packages/client/eslint.config.js Adds new flat config format ESLint configuration

@@ -0,0 +1,22 @@
const globals = require('globals')
Copy link
Preview

Copilot AI Jul 19, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The 'globals' package is imported but not listed as a dependency in package.json. This will cause a runtime error when ESLint tries to load the configuration.

Copilot uses AI. Check for mistakes.

@scholarsmate scholarsmate merged commit 833aed7 into ctc-oss:main Jul 19, 2025
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant