fix: Don't require ESLint

[nzakas]

Prerequisites checklist

• I have read the contributing guidelines.

What is the purpose of this pull request?

Make ESLint an optional peer dependency.

What changes did you make? (Give an overview)

Added peerDependenciesMeta to package.json in order to set eslint as an optional peer dependency.

Practically speaking, the plugin doesn't require ESLint to be useful. We are using it in the Code Explorer without ESLint, where it's installation is causing npm errors because Code Explorer uses ESLint v8.x.

Related Issues

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

It seems like we shouldn't block installation of this package even if ESLint isn't installed or ESLint v8.x is installed, but let me know if anyone has a different opinion. From what I can tell, this change won't have any noticeable effect to most users of this plugin.

[amareshsm]

Marking ESLint as optional is a good idea. Looks good to me. LGTM

[fasttime]

It seems like we shouldn't block installation of this package even if ESLint isn't installed or ESLint v8.x is installed, but let me know if anyone has a different opinion.

I think that to allow installing ESLint v8.x without the --force flag, the peerDependencies version for eslint should be changed to something like "^8.0.0 || ^9.6.0". Currently it's "^9.6.0".

json/package.json

Line 71 in 08fd5d3

"eslint": "^9.6.0",

Making the peer dependency optional will allow a package to install @eslint/json without eslint, but not with a version of eslint which is out of range.

For the same reason in eslint we have the jiti optional peer dependecy version set to "*" and not "^1.21.0", so consumers that use jiti for other purposes than to support eslint are free to install any version they want.

From what I can tell, this change won't have any noticeable effect to most users of this plugin.

Probably most users of this plugin will already have a new versioneslint declared in their devDependencies. So yes, there shouldn't be any noticeable effects. I'm fine with merging this as a fix: 👍

[nzakas]

Making the peer dependency optional will allow a package to install @eslint/json without eslint, but not with a version of eslint which is out of range.

Hmm, not what we want. I wonder if it's best just remove the peer dependency altogether?

[fasttime]

Hmm, not what we want. I wonder if it's best just remove the peer dependency altogether?

Makes sense I think, if it should be possible to install this package irrespective of eslint.

[amareshsm]

LGTM 👍

[fasttime]

LGTM, thanks!