-
Notifications
You must be signed in to change notification settings - Fork 115
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
provide vextab via npm install & require('vetab') #55
Comments
Working on it. :-) |
Okay fine :) |
Will do. Thanks. -- sent frm fone
|
I tried to require vextab via npm, but it doesn't work, because of this line: ff5877a#diff-35f81cb6401704ff630d0f1f39a22707R11 And currently it only works if you use browserify, it would be work with any tool, if you add a prepublish step for npm and publish the js files instead of the coffee files. |
I can do PR with a fix. |
I'm out of town till Apr 2nd. Will take a look when I'm back. -- sent frm fone
|
@0xfe hi, what is the status of this? I'd like to send you PR with js build, so you can publish pure JS module to npm, but I am not able to run tests from master, i have basically this problem: https://travis-ci.org/0xfe/vextab/builds/157201147 |
Same here, requiring vextab requires something that understands coffeescript, as a workaround I just included vextab-div seperate from my bundle... |
@vojty what's about this? |
@SalahAdDin well my PR is almost mergeable, but I need help with tests. Current master branch has some issues #92 maybe @0xfe could help... |
This issue come from here: #53 (comment)
The next goal should be to support other module loader, as I explained here via a UMD: #53 (comment)
The text was updated successfully, but these errors were encountered: