Useless template literal found JS-R1004
Anti-pattern
Minor
a year ago
—
a year old
Template string can be replaced with regular string literal
183 try {
184 let response = await web3.eth.sendSignedTransaction('0x' + signedTx.serialize().toString('hex'))
185 if(response.transactionHash == null) {
186 res.status(500).json({ message: `Transaction not mined` })187 }
188 response.hash = response.transactionHash
189 console.log('log->transaction', response.hash)Template string can be replaced with regular string literal
239 try {
240 let response = await web3.eth.sendSignedTransaction('0x' + signedTx.serialize().toString('hex'))
241 if(response.transactionHash == null) {
242 res.status(500).json({ message: `Transaction not mined` })243 }
244 response.hash = response.transactionHash
245 console.log('log->transaction', response.hash)
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()}`