-
Notifications
You must be signed in to change notification settings - Fork 5
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
publish to npm? #2
Comments
Anyway, maybe should publish |
Thanks - sorry for the delay but I'm out of the county at the moment. At least initially I was waiting to see if there would be any problems with it - but it's been quite quiet and now it seems a good time (although I think I ought to fix the flasher first). Adding the tools would be good too - but we should pull all the crazy module loading stuff from this into an index.js that 'just works' |
Hmm, tried to load an example but I think the node serial stuff is wonky. It returns UInt8Array instances instead of strings when used from this tool. I was hoping to contribute on this project but the necessary workarounds are too painful--jsdom and the like. If someone can extract the necessary communication code into its own library then it'd be more sane. I'd envision a NodeJS module which was then compiled with browserify for the IDE. You could leverage the same library here, the IDE, and anyone else could build whatever they want on its API. I can help make this happen if you're interested. |
Perhaps you're holding off until it's proven itself a bit?
The text was updated successfully, but these errors were encountered: