Skip to content
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

No template literals in normal strings #15

Closed
jfmengels opened this issue Apr 18, 2016 · 10 comments
Closed

No template literals in normal strings #15

jfmengels opened this issue Apr 18, 2016 · 10 comments

Comments

@jfmengels
Copy link
Contributor

eslint/eslint#5850
Do we want to create a rule for this here?
I don't think it happens often but can be useful, and we might as well put it here as we have/want plenty of different type of rules.

@sindresorhus
Copy link
Owner

Yeah. Doesn't have to happen often to be useful.

Might want to wait on this though (from linked issue):

I'll leave this open for a bit to see if anyone on the team wants to disagree.

@ljharb
Copy link

ljharb commented Apr 19, 2016

It's now closed, sadly. I'd love to be able to use this rule.

@sindresorhus
Copy link
Owner

Alright. Let's do it then.

@ljharb
Copy link

ljharb commented Apr 19, 2016

@sindresorhus it'd be awesome if it could be done standalone, and made a dep of this one, so i can use it in other packages without having to bring in this entire package :-)

@jfmengels
Copy link
Contributor Author

jfmengels commented Apr 19, 2016

You wouldn't get much less by having a standalone module (there a not a lot of dependencies... yet), and maybe you'll want all the awesome rules that it has and will have? :D

More seriously, to me, it's a bit of a pain to create a new project for it. I agree that you might not want all the rest, but maintaining-wise, it's simpler to have it in a project that is actively maintained by several people, not in a corner somewhere. I proposed to add it here because this is a collection of various rules, and what you asked for is kind of a specific rule, with no existing plugin (afaik) it could get included in.

Then again, if we write this rule, you're free to copy it in a new repo and use it as you see fit.

@ljharb
Copy link

ljharb commented Apr 19, 2016

I use eslint-find-rules --unused to ensure that all rules available are defined, so if I bring in a plugin with N rules, i have to add N lines to my eslint config.

If the authors don't mind, then once written, I'd be happy to spend the few minutes required to create a new project, and would be happy to copy the code and share ownership/maintainership of it, especially if this module could pull it in as a dep.

@jamestalmage
Copy link
Contributor

More seriously, to me, it's a bit of a pain to create a new project for it.

Maybe we should build this project as a monorepo? Split each rule into its own package? It would be interesting to explore what AVA and XO could do to enable monorepo development:

  • shared config
  • easily test/lint all submodules/packages
  • easily test/lint an individual submodule/package

@jfmengels
Copy link
Contributor Author

jfmengels commented Apr 21, 2016

Maybe we should build this project as a monorepo

Can be interesting.

explore what AVA and XO could do to enable monorepo development

Does AVA already do something monorepo-like, or are you suggesting it could start doing something in that vein? If so, I'm curious as to what parts of it.

@sindresorhus
Copy link
Owner

sindresorhus commented Apr 21, 2016

I'm really not a big fan of monorepos. At this point it would seem like a premature complication. It would also add a lot of overhead. I think the rules we have now makes sense in one package. We currently have a nice setup with tests, docs and code coverage. I don't really see the problem depending on this plugin, even if you only want one rule. We currently only have 4 tiny dependencies. Dependencies matters less in a dev environment.

I use eslint-find-rules --unused to ensure that all rules available are defined, so if I bring in a plugin with N rules, i have to add N lines to my eslint config.

I would rather fix that module. It should have an ignore option.

@jfmengels
Copy link
Contributor Author

Closing as it's being integrated in ESLint core rules eslint/eslint#6767

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants