Disable backtick quotes if it is not a template

[a-rodin]

Now you could write either

let a = 'hello'

or

let a = `hello`

And it would pass standard linter for both cases, even if in the second case the template doesn't have any interpolated variables and in fact is just a different form of a string. So you could mix both forms, something conceptually similar to mixing " and ' quotes.

ESLint allows us to prevent this behavior by adding a rule

"quotes": [2, "single"]

This rule makes ESLint to throw an error for this

let a = `hello`

but don't throw any errors for this

let subject = 'world'
let a = `hello, ${world}`

Also see this issue in ESLint.

[dcousens]

NACK, only because it means if I am intermittently using templates, I have more rules to remember or otherwise have standard bark at me.
Reminds me of #791

[yoshuawuyts]

I don't mind this, it'd warn for weird mixing stuff like:

var a = `hello` + '  my ' + `brethren`

Standard would tell you that you probably meant to either use single quotes or interpolate, but definitely not the weird thing right there above. I feel that this would indeed be in the spirit of our current approach to #791. There's a default way to do things, and we warn if people are deviating from it without there being a need to.

note: I don't really mind whatever way it goes, just like - feel it's consistent with the ruleset we've been applying so far.

edit: Also consider this:

var foo = 'hello people' // ✓
var bar = `nope, or dope? who knows` // ✓
var baz = "vinegar sauce" // 𝗑 - not sure why

[dcousens]

it'd be simpler to just disable non-backtick quotes entirely

[a-rodin]

@dcousens In this case it would be necessary to update README.md and consequently the front page of https://standardjs.com , as they both mandate to use

Single quotes for strings – except to avoid escaping

[skbolton]

Yeah personally I use back ticks all the time that way I am always ready to interpolate

[feross]

it'd be simpler to just disable non-backtick quotes entirely

I like having two types of quotes to indicate intent. Interpolating is rare enough that I appreciating having it explicitly called out by the ` character.

[skbolton]

Well really there is no harm leaving a template string as an un-interpolated string. Yeah normally it means that you are going to interpolate but I think it would be weird to have that as a rule

[a-rodin]

There are also a couple of other arguments against making backtick quotes preferred for all strings:

1. Using backticks for strings without interpolated variables adds a burden of additional care about escaping, because if you are not planning to interpolate, but for some reason want to have in your string literally ${console}, then it should be properly escaped in order to avoid runtime errors or perhaps even vulnerabilities. It is also not always possible for any linter to automatically determine the cases when an expression should be escaped and when it is not, like in this example with ${console}.
2. Making backticked strings to be the only possible strings to use instead of single quoted ones would break all the codebases in pure ES5 that can't use string template syntax, but use standard to enforce style rules.

[blgm]

I can see how disabling backticks would fit with Standard's existing philosophy

• In ES5, the the rule is that you must use single quotes, except when you need the extra functionality of double quotes (e.g. to quote a single quote)
• In ES6, a rule saying that you must use single quotes, except when you need the extra functionality of double quotes or backticks would make sense. Extra functionality would include:
  • quoting other types of quote
  • templates
  • literal new lines

[romanzoller]

I also think this would make sense, such that

Single quotes for strings – except to avoid escaping

is enforced. Note that the implementation is very simple: Remove the "allowTemplateLiterals": true option from the quotes rule. Despite the naming of the option, "allowTemplateLiterals": false actually still allows template literals when their extra functionality is used, see examples in the doc for the quotes rule.

[mightyiam]

standard/eslint-config-standard#151

[feross]

Thanks for the input everyone!

I think that we should disallow template literals when placeholders or tagged template features are not used.

console.log('hello there')    // ✓ ok
console.log("hello there")    // ✗ avoid
console.log(`hello there`)    // ✗ avoid

$("<div class='box'>")        // ✓ ok
console.log(`hello ${name}`)  // ✓ ok

Only 1 ecosystem package (webtorrent-desktop) needed to be updated, and it was just one line that needed the update: webtorrent/webtorrent-desktop@c8f1e23

I'm going to ship this in v14.

[mightyiam]

Lovely!