Skip to content

ESLint: no-template-curly-in-string rule not found #543

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
suchipi opened this issue Sep 2, 2016 · 7 comments
Closed

ESLint: no-template-curly-in-string rule not found #543

suchipi opened this issue Sep 2, 2016 · 7 comments

Comments

@suchipi
Copy link

suchipi commented Sep 2, 2016

Just upgraded from 0.3.0-alpha(via git url; wanted jest early) to 0.3.0. Using Atom's linter-eslint, I now see this ESLint warning in all of my files:

[ESLint] [Warning] [no-template-curly-in-string] Definition for rule 'no-template-curly-in-string' was not found at line 1 col 1 in src/whatever.js

This doesn't show up in the output from npm start.
I checked the README to see if I needed to install any new eslint plugins globally, but there aren't any more there than what I already had.

linter-eslint is configured to use my global ESLint installation.

What do I need to do to get this rule installed?

@gaelollivier
Copy link
Contributor

gaelollivier commented Sep 2, 2016

Have the same problem.

We need this PR to be merged in fast-eslint I guess.

@gaearon
Copy link
Contributor

gaearon commented Sep 2, 2016

Hmm, wouldn’t updating the global ESLint to the latest version solve the issue?

@gaelollivier
Copy link
Contributor

gaelollivier commented Sep 2, 2016

Nope, I tried and it doesn't. fast-eslint probably uses its own version of eslint

@gaearon
Copy link
Contributor

gaearon commented Sep 2, 2016

I pinged @Adezandee. In the meantime you may try using a different ESLint atom plugin that uses the global version of ESLint instead of shipping its own copy.

@arnaud-dezandee
Copy link

arnaud-dezandee commented Sep 2, 2016

I'll update eslint v3.4.0 inside fast-eslint asap. 👍
@gaelduplessix, Indeed the eslint engine is built-in. Everything else is provided by the environment, either project or globally installed modules.
See https://github.com/Adezandee/fast-eslint/blob/master/lib/exec.js#L26-L39

@gaearon
Copy link
Contributor

gaearon commented Sep 2, 2016

Thanks for quick turnaround @Adezandee, it is very much appreciated.

@arnaud-dezandee
Copy link

arnaud-dezandee commented Sep 2, 2016

I released fast-eslint v3.1.0. I bootstraped a pristine react app with the CLI to try things out. With npm@3 flattening it should work right away. With npm@2, you'll probably have to install some deps globally since fast-eslint isn't smart enough to add relative reference inside the lookup array for modules loading (it should add ./node_modules/react-scripts/node_modules to the array).
Could be done though :)

@gaearon gaearon closed this as completed Sep 2, 2016
@lock lock bot locked and limited conversation to collaborators Jan 22, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants