react/require-extension is deprecated

[tizmagik]

The react/require-extension rule is deprecated. Please use the import/extensions rule from eslint-plugin-import instead.

[cwonrails]

This is an upstream issue with eslint-plugin-import. A temporary solution was posted here: airbnb/javascript#978 (comment).

I'd be glad to put in a PR. I've also updated the outdated deps in package.json in my fork, but I can make a clean branch with this change alone.

[ccpricenytimes]

A PR with the change would be awesome @cwonrails Thanks!

[cwonrails]

No problem at all! Incoming now 👍

[tizmagik]

Hmm, well we wouldn't want to 'lose' out on this rule necessarily. It's still useful even if it's deprecated. That workaround seems to suggest to disable the rule just to remove the warning that gets logged. I'm not sure that makes sense. Thoughts?

[cwonrails]

Understood. The workaround is to turn it off for that error while also adding the non-deprecated one.

[tizmagik]

Ah, gotcha, that sounds good then! 👍 Thanks!

[cwonrails]

Of course! PR here: #165