Skip to content

Upgrade eslint to latest (with flat config update)#1283

Merged
itsalaidbacklife merged 2 commits intomainfrom
feat/eslint-upgrade
Oct 30, 2025
Merged

Upgrade eslint to latest (with flat config update)#1283
itsalaidbacklife merged 2 commits intomainfrom
feat/eslint-upgrade

Conversation

@seriouslysean
Copy link
Copy Markdown
Collaborator

@itsalaidbacklife depending on which of these PRs merge first, I'll probably have to rejigger a few things to align the dependencies. See these:

Issue number

Relevant issue number

  • Resolves #

Please check the following

  • Do the tests still pass? (see Run the Tests)
  • Is the code formatted properly? (see Linting (Formatting))
  • For New Features:
    • Have tests been added to cover any new features or fixes?
    • Has the documentation been updated accordingly?

Please describe additional details for testing this change

@seriouslysean seriouslysean self-assigned this Oct 29, 2025
@seriouslysean seriouslysean added enhancement New feature or request hacktoberfest This issue welcomes contributions for Hacktoberfest. version-minor An update that warrants a bumping the project's minor version (e.g. 4.0.0 => 4.1.0) labels Oct 29, 2025
@seriouslysean seriouslysean changed the title Upgrade eslint Upgrade eslint to latest (with flat config update) Oct 29, 2025
Copy link
Copy Markdown
Contributor

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 v8.54.0 to v9.38.0 and migrates to the flat configuration format, addressing the deprecation of the legacy .eslintrc.js format. The changes include updating ESLint plugins, refactoring the configuration file, and applying formatting fixes to ensure code compliance with the new linter setup.

  • Upgraded ESLint to v9.38.0 and related plugins to compatible versions
  • Migrated from .eslintrc.js to the new eslint.config.js flat config format
  • Applied formatting fixes across the codebase for compliance with updated linting rules

Reviewed Changes

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

Show a summary per file
File Description
package.json Updated ESLint and related plugin dependencies to v9-compatible versions
package-lock.json Locked dependency versions reflecting the ESLint ecosystem upgrade
eslint.config.js New flat config file replacing the deprecated .eslintrc.js format
.eslintrc.js Removed legacy ESLint configuration file
vitest.config.mjs Applied formatting fixes for array bracket spacing
tests/e2e/support/helpers.js Reformatted multi-line conditional for readability
tests/e2e/plugins/index.js Removed unused eslint-disable comment
src/stores/game.js Refactored computed properties for improved readability and formatting
.storybook/preview.js Updated quotes to single quotes per linting rules
.storybook/main.js Applied formatting fix for array bracket spacing

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@itsalaidbacklife itsalaidbacklife added version-patch An update that warrants a bumping the project's patch version (e.g. 4.0.0 => 4.0.1) and removed version-minor An update that warrants a bumping the project's minor version (e.g. 4.0.0 => 4.1.0) labels Oct 30, 2025
@itsalaidbacklife itsalaidbacklife merged commit f9cc8a5 into main Oct 30, 2025
9 checks passed
@itsalaidbacklife itsalaidbacklife deleted the feat/eslint-upgrade branch February 10, 2026 19:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request hacktoberfest This issue welcomes contributions for Hacktoberfest. version-patch An update that warrants a bumping the project's patch version (e.g. 4.0.0 => 4.0.1)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants