Skip to content

Conversation

wcandillon
Copy link
Owner

No description provided.

@Copilot Copilot AI review requested due to automatic review settings September 21, 2025 16:50
Copy link

@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 migrates the project from ESLint v7 to v9 configuration system and updates repository URLs. The primary purpose is to modernize the linting configuration and correct repository references.

  • Migrates from legacy .eslintrc to the new flat config system with eslint.config.mjs
  • Updates ESLint from v7.8.1 to v9.0.0 and corresponding config package
  • Corrects repository URLs from redash to react-native-redash throughout the project
  • Removes outdated ESLint disable comments that are no longer needed
  • Updates CI badge from CircleCI to GitHub Actions

Reviewed Changes

Copilot reviewed 10 out of 12 changed files in this pull request and generated no comments.

Show a summary per file
File Description
eslint.config.mjs New flat ESLint configuration file replacing legacy format
.eslintrc Removed legacy ESLint configuration file
package.json Updated ESLint dependencies and corrected repository URLs
README.md Updated CI badge from CircleCI to GitHub Actions
src/tests/paths.ts Removed unnecessary max-len ESLint disable comment
src/tests/matrix.ts Removed no-shadow ESLint disable, renamed parameters, improved formatting
src/tests/Paths.test.ts Removed unnecessary max-len ESLint disable comments
src/Paths.ts Added trailing comma for consistency
src/Matrix4.ts Added trailing comma for consistency
src/Matrix3.ts Added trailing comma for consistency

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@wcandillon wcandillon merged commit b4b4a92 into master Sep 21, 2025
1 check 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