Warning: React version not specified in eslint-plugin-react settings.

[kud]

Hello,

Since I've updated this plugin, i've got a message like Warning: React version not specified in eslint-plugin-react settings..

This is true, I didn't specify the version of react, and this happens due to 8738e59#diff-4e51c5c91adee02670adaca34f7fd7fdR17

However, when I read the doc, I've got this:

"version": "15.0", // React version, default to the latest React stable release

It's said "by default, it uses the latest version of react".

So why displaying a warning if there's a default and this is what I want? as I keep my react often upgraded.

[ljharb]

(linking to your comment here: #1857 (comment))

[ljharb]

Just because there's a default doesn't mean you shouldn't always be providing an explicit value.

Repeating #1857 (comment):

i could see adding a separate option for "detect" (using the "version" as a fallback), which did require('react').version - want to open a PR that adds that?
(Note that it'd have to use process.cwd(), and the resolve package, to get the proper require path)

[kud]

Yes, it's true.

For the moment, I use this workaround: "version": "999.999.999".

[chrisdothtml]

+1 for supporting latest as the value

[marcofugaro]

Yeah please I don't want to update this config each time a new react comes out

[ljharb]

You have to update your package.json every time that happens; I’m not sure also updating eslintrc is much harder.

[marcofugaro]

Yeaaah... I don't have it in a package.json, I have it in the eslint config shared for the team, and each time I would have to upgrade deps, update by hand, publish a new version of the tool.

Most importantly, I upgrade my dependencies using yarn upgrade-interactive --latest, not by hand. I (or other mantainers) might forget to update this version by hand.

[chrisdothtml]

We also use a shared eslint config and this would make it quite annoying every upgrade. If "latest" isn't supported, everyone's just gonna use "999.999.999" instead, which could lead to issues in the future if this plugin assumes that property contains their actual React version

[kud]

#1955 (comment)

Well, I use ncu to update my packages, and yes it's not hard to update .eslintrc but you have to remember this and it's not something I want. :D

I'm using 999.999.999 for the moment.

[bhardy]

Comment removed and new issue created: #1963

[ljharb]

@bhardy that seems like a different issue; if you have the settings object with a react version you shouldn't get a warning at all.

[bhardy]

@ljharb would you like me to create a new issue?

[ljharb]

@bhardy yes please :-)