JSRocksHQ / harmonic

The next static site generator
http://harmonicjs.com/
MIT License
282 stars 26 forks source link

Unable to use template strings #94

Closed jaydson closed 9 years ago

jaydson commented 9 years ago

Unfortunately, we can't use template strings due to jscs issues. We have a lot of code concatenation and we can improve the code by using template strings.

What can we do? Disable jscs?

Suggestions are welcome.

//cc @leobalter @UltCombo

UltCombo commented 9 years ago

What do you mean? Does JSCS crash or just emit an warning? If it is just an warning (as reported in this issue) then we can most likely suppress the warning with JSCS comments.

jaydson commented 9 years ago

Yep, it is just a warning message, but the message doesn't make any sense in the context. So, you mean we can disable JSCS entirely per block?

Example:

/* jscs ignore:start */
let bar = 'bar';
let foo = `foo ${bar}`;
/* jscs ignore:end */

PS: I don't know if this ignore is valid

UltCombo commented 9 years ago

I'd suggest more like

UltCombo commented 9 years ago

Though, if we are going to use a lot of template strings, there would be too much comment litter.

leobalter commented 9 years ago

Nothing else to say after @UltCombo but how I liked the issue reference above the ignore lines. That's awesome to keep a minimum track for external projects bugs in the code.

jaydson commented 9 years ago

Solved by #109

UltCombo commented 9 years ago

Nice! :+1: