Should we use thedefineConfig(a, b, c) ordefineConfig([a, b, c]) style? The eslint docs (inexplicably) usedefineConfig([a, b, c]) throughout, butdefineConfig(a, b, c) is much better in a TS context since each argument's type is checked individually. I've gone with that approach for now, but we can change to match their style if we want.

kirkwaiblinger added6 commits

August 25, 2025 18:37

WIP - deprecate tseslint.config()

533d973

yup

2e44d3f

gud

3fff324

issue

4d73b73

switch docs to use defineConfig

e81ad55

add back import

7fb7e71

Copy link

Contributor

typescript-eslintbot commentedAug 26, 2025

Thanks for the PR,@kirkwaiblinger!

typescript-eslint is a 100% community driven project, and we are incredibly grateful that you are contributing to that community.

The core maintainers work on this in their personal time, so please understand that it may not be possible for them to review your work immediately.

Thanks again!

🙏Please, if you or your company is finding typescript-eslint valuable, help us sustain the project by sponsoring it transparently on https://opencollective.com/typescript-eslint.

Copy link

netlifybot commentedAug 26, 2025•
edited
Loading

✅ Deploy Preview fortypescript-eslint ready!

Name	Link
🔨 Latest commit	`1576c1f`
🔍 Latest deploy log	https://app.netlify.com/projects/typescript-eslint/deploys/68b52f48176ffd0008b877c8
😎 Deploy Preview	https://deploy-preview-11531--typescript-eslint.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.
Lighthouse	1 paths audited Performance: 82 (🔴 down 15 from production) Accessibility: 97 (no change from production) Best Practices: 100 (no change from production) SEO: 92 (no change from production) PWA: 80 (no change from production) View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to yourNetlify project configuration.

Copy link

nx-cloudbot commentedAug 26, 2025•
edited
Loading

View yourCI Pipeline Execution ↗ for commit1576c1f

Command	Status	Duration	Result
`nx run-many -t lint`	✅ Succeeded	3m 28s	View ↗
`nx run-many -t typecheck`	✅ Succeeded	2m	View ↗
`nx test typescript-estree --coverage=false`	✅ Succeeded	2s	View ↗
`nx test eslint-plugin --coverage=false`	✅ Succeeded	3s	View ↗
`nx test eslint-plugin-internal --coverage=false`	✅ Succeeded	2s	View ↗
`nx run types:build`	✅ Succeeded	5s	View ↗
`nx run integration-tests:test`	✅ Succeeded	3s	View ↗
`nx run generate-configs`	✅ Succeeded	8s	View ↗
`Additional runs (27)`	✅ Succeeded	...	View ↗

☁️Nx Cloud last updated this comment at2025-09-01 05:35:32 UTC

kirkwaiblinger added5 commits

August 26, 2025 23:39

lint

395c24b

more spots

4bad07b

typo

50dd8ba

#config-deprecated

b0de934

touchup

2403962

kirkwaiblinger changed the title~~docs: deprecate tseslint.config()~~feat: deprecate tseslint.config()

Aug 27, 2025

kirkwaiblinger marked this pull request as ready for review

August 27, 2025 17:28

JoshuaKGoldberg reviewed

Aug 27, 2025

View reviewed changes

docs/getting-started/Quickstart.mdxShow resolvedHide resolved

JoshuaKGoldberg previously approved these changes

Aug 27, 2025

View reviewed changes

Copy link

Member

JoshuaKGoldberg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

Always nice to see a formatter issue filed as a result of docs work 😅

Great stuff!

docs/getting-started/Quickstart.mdx OutdatedShow resolvedHide resolved

docs/packages/TypeScript_ESLint.mdx OutdatedShow resolvedHide resolved

Update docs/getting-started/Quickstart.mdx

ce6cda6

Co-authored-by: Josh Goldberg ✨ <git@joshuakgoldberg.com>

kirkwaiblinger dismissedJoshuaKGoldberg’sstale review viace6cda6

August 27, 2025 20:37

Update docs/packages/TypeScript_ESLint.mdx

18a76fc

Co-authored-by: Josh Goldberg ✨ <git@joshuakgoldberg.com>

kirkwaiblinger requested a review fromJoshuaKGoldberg

August 27, 2025 20:38

Josh-Cena approved these changes

Aug 28, 2025

View reviewed changes

Copy link

Member

Josh-Cena left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

LGTM

Josh-Cena added the 1 approval>=1 team member has approved this PR; we're now leaving it open for more reviews before we merge label

Aug 28, 2025

kachkaev mentioned this pull request

Aug 28, 2025

Closed

kirkwaiblingerenabled auto-merge (squash)

September 1, 2025 05:29

Merge branch 'main' into deprecate-tseslint-config

1576c1f

kirkwaiblinger merged commitd7614a7 intotypescript-eslint:main

Sep 1, 2025

63 checks passed

kirkwaiblinger deleted the deprecate-tseslint-config branch

September 1, 2025 15:22

chrisbbreuer mentioned this pull request

Sep 2, 2025

chore(deps): update all non-major dependenciesstacksjs/eslint-plugin#54

Closed

Copy link

stefanprobst commentedSep 2, 2025•
edited
Loading

is theConfigWithExtends type available publicly?

stevezhu mentioned this pull request

Sep 3, 2025

Re-export eslint/config exportsstevezhu/eslint-config#44

Merged

stevezhu added a commit to stevezhu/eslint-config that referenced this pull request

Sep 3, 2025

Re-export eslint/config exports (#44)

01180a7

typescript-eslint/typescript-eslint#10935typescript-eslint/typescript-eslint#11531

Copy link

corneliusroemer commentedSep 4, 2025

Migration guide for deprecatedtseslint.config() ->defineConfig:https://typescript-eslint.io/packages/typescript-eslint/#migrating-to-defineconfig

Copy link

MemberAuthor

kirkwaiblinger commentedSep 4, 2025

@stefanprobst

is theConfigWithExtends type available publicly?

This isn't directly related to this PR, and we anyways try to avoid commenting on closed PRs (docs). Would you mind reaching out to us with one of the methods in the contributing guide to continue the conversation?

Thanks!