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

Local script tag source should start with file:// #23

Merged
merged 1 commit into from
Dec 8, 2017

Conversation

warpdesign
Copy link
Contributor

For some reason, it worked in Unix, but did not under Windows where the script silentely failed to be loaded and later produced the error $ is not a function.

This makes sure the script is loaded in both Windows & Unix and fixes #16

For some reason, it worked in Unix, but did not under Windows where the script silentely failed to be loaded and later produced the error `$ is not a function`.

This makes sure the script is loaded in both Windows & Unix and fixes onury#16
@onury
Copy link
Owner

onury commented Dec 8, 2017

@warpdesign, sure! Thanks for this PR.

@onury onury merged commit 1880eda into onury:master Dec 8, 2017
@onury
Copy link
Owner

onury commented Dec 8, 2017

You have a syntax error though. Will fix it before releasing.

@warpdesign
Copy link
Contributor Author

Oh, sorry :/

@onury
Copy link
Owner

onury commented Dec 8, 2017

No problem at all. v1.5.2 is released.
BTW, this will be better addressed in v2 via path.join(__dirname, ...) instead of file:// protocol.
Thanks.

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

Successfully merging this pull request may close these issues.

$ is not a function
2 participants