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

ignore:line option has no effect on scripts with type of text/template #188

Closed
binarykitchen opened this issue May 21, 2017 · 1 comment
Closed

Comments

@binarykitchen
Copy link

this is probably a rare case. developing a wordpress plugin here using a html file as a template to inject an inline script which works fine. the only problem is that js-standard in my atom editor is complaining with this warning

"Expected an assignment or function call and instead saw an expression. (no-unused-expressions)"

understand, it's because of the {{{ part which makes sense. but the problem is that the // jshint ignore:line part at end of this line does not seem to have any effect.

<script id='tmpl-nf-field-videomail' type='text/template'>
  <div id='nf-field-{{{ data.id }}}' name='nf-field-{{{ data.id }}}' class='{{{ data.classes }}} nf-element videomail'> // jshint ignore:line
   ...
</script>

maybe this exotic use case isn't covered in standardjs yet?

@sonicdoe
Copy link
Collaborator

Since this ultimately revolves around us mistakenly linting non-JavaScript inline scripts, I’ll close this in favor of #182.

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

No branches or pull requests

2 participants