feat!: update minimatch

[Pixel998]

Prerequisites checklist

• I have read the contributing guidelines.

What is the purpose of this pull request?

This PR updates the minimatch package to the latest version

What changes did you make? (Give an overview)

• Updated minimatch to v10.0.3
• Switched to named imports for minimatch, in line with the updated API.
• Removed the deprecated @types/minimatch package since minimatch now includes its own built-in TypeScript type definitions.

minimatch v10 requires Node.js v20 or higher, so marking this PR as draft until we drop Node.js v18.

Related Issues

Fixes #230
Fixes #66

Is there anything you'd like reviewers to focus on?

[mdjermanovic]

Can we now remove allowWindowsEscape: true, option, since that behavior should be on by default in minimatch v10?

rewrite/packages/config-array/src/config-array.js

Line 65 in 88b0c34

allowWindowsEscape: true,

[mdjermanovic]

We should also update minimatch dependency in the eslint package:

https://github.com/eslint/eslint/blob/e1ac05e2fae779e738f85bd47dda1cc2b7099346/package.json#L139

[Pixel998]

We should also update minimatch dependency in the eslint package

Yeah, it’s on my todo list — I’ll get to it today or tomorrow.

[aladdin-add]

Yeah, it’s on my todo list — I’ll get to it today or tomorrow.

@Pixel998 I can take it(if you don't have time to look at it.)🙌

[Pixel998]

@aladdin-add Sorry for the late reply! It looks like there’s already a PR open for it.

[fasttime]

When this is PR is merged, we should do a release and then update the @eslint/config-array dependency in the eslint repo in eslint/eslint#20246. I've updated the migration guide in that PR, so please check if my text makes sense, and feel free to change it.

[fasttime]

LGTM, thanks! Leaving open for @mdjermanovic to review.

[mdjermanovic]

I think we can remove allowWindowsEscape: true (#273 (comment)). It's a legacy, now-undocumented option (although not explicitly deprecated).

[Pixel998]

@mdjermanovic I thought I had already made that change, but I must have missed it. Sorry about that!

[mdjermanovic]

LGTM, thanks! Leaving open for @fasttime to verify last changes.