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

Minor changes to make it compatible with both WSH and ASP #449

Closed
wants to merge 1 commit into from

Conversation

nagaozen
Copy link
Contributor

<script>, even in comments, triggers Active Server Pages error 'ASP 0138', Nested Script Block Error.

Also, using global ← this pattern works on WSH but not on ASP, it's better to just Mustache = {}, since its less typing, works on all platforms and it's the the desired behavior.

`<script>`, even in comments, triggers `Active Server Pages error 'ASP 0138', Nested Script Block` Error.

Also, using `global ← this` pattern works on WSH but not on ASP, it's better to just Mustache = {}, since its less typing, works on all platforms and it's the the desired behavior.
@dasilvacontin
Copy link
Collaborator

Hi @nagaozen, thanks for the PR!

Unfortunately, it's breaking the Travis build: the linter quits. Can you fix it? Cheers!

> eslint mustache.js
mustache.js
  14:12  error  "Mustache" is not defined  no-undef

@phillipj
Copy link
Collaborator

Fixed with 60c2da8

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

Successfully merging this pull request may close these issues.

3 participants