Skip to content

feat(deps): test eslint-config-typescript import-x prerelease - #508

Draft
michaeljaltamirano wants to merge 4 commits into
mainfrom
t/CPL-1863_test_eslint-config-typescript-import-x-prerelease
Draft

michaeljaltamirano wants to merge 4 commits into
mainfrom
t/CPL-1863_test_eslint-config-typescript-import-x-prerelease

Conversation

@michaeljaltamirano

@michaeljaltamirano michaeljaltamirano commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

Michael note

tsup was evaluated as a replacement for tsc, but I think there are benefits to tsdown, despite tsdown not yet having a v1 release:

Summary

  • Test PR only — not meant to merge as-is. Bumps @open-turo/eslint-config-typescript to the eslint-plugin-import-x migration prerelease (24.0.0-pr-591.722.1.1) to validate the breaking change in a real consumer repo before the v24 release ships.
  • Renames the three import/no-default-export disable comments in this repo to import-x/no-default-export per the migration guide.
  • Regenerates the jest snapshot to reflect the import-x/* rule keys.

Test plan

  • npm run tsc passes
  • npm run lint passes clean (no missing-rule errors, no new violations from the two rules eslint-plugin-import didn't have)
  • npm test passes with regenerated snapshot
  • Once eslint-config-typescript v24 is actually released, swap this to the real version (or close this PR — it exists purely to validate the prerelease)

@michaeljaltamirano michaeljaltamirano added ai-review Triggers AI code review bot relay PR created by the relay workflow labels Aug 7, 2026
@github-actions

github-actions Bot commented Aug 7, 2026

Copy link
Copy Markdown

Breaking changes file docs/breaking-changes/v25.md

Breaking changes in v25

  • Replaces eslint-plugin-import with eslint-plugin-import-x
    • import/* rule and disable-comment prefixes must be renamed to import-x/* (e.g. // eslint-disable-next-line import/no-cycle// eslint-disable-next-line import-x/no-cycle)
    • The import/resolver setting key is now import-x/resolver
    • config.plugins.import now refers to the eslint-plugin-import-x plugin object

Testing the eslint-plugin-import -> eslint-plugin-import-x migration
(open-turo/eslint-config-typescript#591) as a consumer, ahead of the
v24 release. Renames import/no-default-export disable comments to
import-x/no-default-export to match.

Ref: CPL-1863

BREAKING CHANGE: This commit replaces eslint-plugin-import with eslint-plugin-import-x
@michaeljaltamirano
michaeljaltamirano force-pushed the t/CPL-1863_test_eslint-config-typescript-import-x-prerelease branch from 39d6008 to 685b2ab Compare August 7, 2026 19:32
@michaeljaltamirano michaeljaltamirano changed the title chore(deps): test eslint-config-typescript import-x prerelease feat(deps): test eslint-config-typescript import-x prerelease Aug 7, 2026
@open-turo-bot

Copy link
Copy Markdown
Contributor

Prerelease build

Build version: 25.0.0-pr-508.571.1.1
Docker image: ``

Build output

@open-turo-bot

Copy link
Copy Markdown
Contributor

Prerelease build

Build version: 25.0.0-pr-508.573.2.1
Docker image: ``

Build output

@open-turo-bot

Copy link
Copy Markdown
Contributor

Prerelease build

Build version: 25.0.0-pr-508.574.1.1
Docker image: ``

Build output

@michaeljaltamirano
michaeljaltamirano force-pushed the t/CPL-1863_test_eslint-config-typescript-import-x-prerelease branch from abb6f8d to c2afb51 Compare August 10, 2026 22:22
@open-turo-bot

Copy link
Copy Markdown
Contributor

Prerelease build

Build version: 25.0.0-pr-508.575.1.1
Docker image: ``

Build output

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ai-review Triggers AI code review bot prerelease relay PR created by the relay workflow

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants