Useless template literal found JS-R1004
Anti-pattern
Minor
5 months ago
—
5 months old
Template string can be replaced with regular string literal
42 error.message ==
43 `itr != _by_name.end(): no method with name 'i_like_turtles'` || // pre-appbase
44 error.message ==
45 `method_itr != api_itr->second.end(): Could not find method i_like_turtles`,46 ); // appbase
47
48 const info = VError.info(error);Template string can be replaced with regular string literal
40 assert.equal(error.name, "RPCError");
41 assert(
42 error.message ==
43 `itr != _by_name.end(): no method with name 'i_like_turtles'` || // pre-appbase44 error.message ==
45 `method_itr != api_itr->second.end(): Could not find method i_like_turtles`,
46 ); // appbase
Description
Template literals are useful when you need:
Strings that have unescaped double quotes and single quotes.
Strings that need line breaks in them.
If neither of these three conditions is met, you can replace the template expression with a regular string literal.
Bad Practice
const dialogue = `"Journey before destination", said Dalinar.`
const dialogue2 = `What is a 'Kwisatz Haderach'?`
Recommended
const dialogue = '"Journey before destination", said Dalinar.'
const dialogue2 = "What is a 'Kwisatz Haderach'?"
const dialogue3 = `"${getLine()}", said ${getChararcter()}`