Skip to content

chore: update eslint to v10 and @ionic/eslint-config to v0.5.0 - #32

Open
ItsChaceD wants to merge 3 commits into
mainfrom
feat/RMET-4734/eslint-10
Open

ItsChaceD wants to merge 3 commits into
mainfrom
feat/RMET-4734/eslint-10

Conversation

@ItsChaceD

@ItsChaceD ItsChaceD commented Sep 2, 2026

Copy link
Copy Markdown
Contributor

Description

The migrator now moves plugins to ESLint 10 and @ionic/eslint-config v1: bumps both dependencies, replaces the eslintConfig block with an eslint.config.cjs, and drops --ext ts from lint scripts since flat config ignores it. The repo itself moves to the same setup.

Flat config has no equivalent of .eslintignore, so the generated config carries the plugin's existing ignore entries over instead of imposing a fixed list. Without that, a plugin that excluded website/, docs/ or a nested package would silently start linting it — on one repo in the rollout that was 15 entries and a flood of new reports. Directory entries become **/name/** (or path/** when nested), file entries are kept as-is, and a leading dot is not mistaken for a file extension. Plugins with no .eslintignore get the previous defaults.

Any leftover .eslintrc* is also deleted, not just .eslintignore — ESLint 10 reads neither, so leaving one behind is dead config.

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

Depends on @ionic/eslint-config@1.0.0 being published (lockfile update pending).

@ItsChaceD
ItsChaceD force-pushed the feat/RMET-4734/eslint-10 branch from c4e99c7 to 3eb0668 Compare September 9, 2026 14:37
@ItsChaceD ItsChaceD changed the title feat: update plugins to ESLint 10 chore: update eslint to v10 and @ionic/eslint-config to v0.5.0 Sep 9, 2026
@ItsChaceD
ItsChaceD marked this pull request as ready for review September 14, 2026 16:05
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