Skip to content

feat(vue-vite): upgrade to eslint@10 and flat config - #1907

Open
TheCrazyMaffin wants to merge 1 commit into
ionic-team:mainfrom
TheCrazyMaffin:vue-vite-eslint-10
Open

TheCrazyMaffin wants to merge 1 commit into
ionic-team:mainfrom
TheCrazyMaffin:vue-vite-eslint-10

Conversation

@TheCrazyMaffin

Copy link
Copy Markdown

Description

  • Migrate to eslint flat config.
    • Config from .eslintignore and .eslintrc.cjs is included, matching the previous functionality.
    • Patterns from .eslintignore that are omitted shouldnt make a difference because they werent files being linted in the first place.
  • Upgrade eslint@^8.35.0 --> ^10.12.0
  • Upgrade eslint-plugin-vue@^9.9.0 --> ^10.11.1
  • Upgrade @vue/eslint-config-typescript@^12.0.0 --> ^14.9.0
  • Add globals@^17.13.0 and @eslint/js@^10.0.1 as required by flat config

Tests or Reproductions

  • Running the test scripts specified in ionic.start.json for the vue-vite templates (blank, list, sidemenu, tabs) finishes without errors.
  • Running eslint for the built templates with inserted deliberate problems detects these problems.

Notes / Comments

  • Currently globals.node are being used but probably arent available in browsers. Example: Using process.env.NODE_ENV in a template does not trigger a eslint problem. (Which would be the case for globals.browser)

@TheCrazyMaffin
TheCrazyMaffin requested a review from a team as a code owner October 6, 2026 08:51
@TheCrazyMaffin
TheCrazyMaffin requested review from brandyscarney and removed request for a team October 6, 2026 08:51
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