Skip to content

feat(cli): migrate apps to ESLint 10 in cap migrate - #8585

Closed
ItsChaceD wants to merge 2 commits into
nextfrom
feat/RMET-4734/cap-migrate-eslint
Closed

feat(cli): migrate apps to ESLint 10 in cap migrate#8585
ItsChaceD wants to merge 2 commits into
nextfrom
feat/RMET-4734/cap-migrate-eslint

Conversation

@ItsChaceD

@ItsChaceD ItsChaceD commented Sep 2, 2026

Copy link
Copy Markdown
Contributor

Description

npx cap migrate can now optionally update apps that use @ionic/eslint-config: bumps ESLint to v10 and the config to v1, replaces the eslintConfig block and .eslintignore with an eslint.config.cjs, and drops --ext ts from scripts. Opt-in only: an "Update ESLint setup? (y/N)" prompt (default no), or the --update-eslint flag for non-interactive runs. When skipped or absent, nothing changes. A block with custom rules is kept so they can be ported manually.

Change Type

  • Fix
  • Feature
  • Refactor
  • Breaking Change
  • Documentation
  • Other (CI, chores, etc.)

Rationale / Problems Fixed

https://outsystemsrd.atlassian.net/browse/RMET-4734

Platforms Affected

  • Android
  • iOS
  • Web

Notes

Should not ship in a release before @ionic/eslint-config@1.0.0 is published.

@ItsChaceD
ItsChaceD marked this pull request as ready for review September 2, 2026 15:37
@ItsChaceD
ItsChaceD marked this pull request as draft September 2, 2026 15:44
@jcesarmobile

Copy link
Copy Markdown
Member

I don't think we should do this for user apps, and I doubt any apps is using @ionic/eslint-config, that package is meant for npm tooling, I don't think it plays well with web code, we don't include it in the apps created by npm init @capacitor/app, nor by apps created with @ionic/cli (they use eslint rules for react, vue or angular based on the chosen framework), and on plugins created with npm init @capacitor/plugin, since it also creates an app with npm init @capacitor/app in the process, we exclude the generated example-app from eslint and prettier as they cause issues on them.

@ItsChaceD ItsChaceD closed this Sep 9, 2026
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.

2 participants