search

machinekit / machinetalk-protobuf

Protobuf declarations for machinekit messages
MIT License
10 stars 11 forks source link

Added JavaScript bundles #49

Closed machinekoder closed 8 years ago

machinekoder commented 8 years ago

This PR adds build script and the actual bundles to use machinetalk-protobuf in the browser.

machinekoder commented 8 years ago

@bobvanderlinden Can you please check if I did not break the npm package

bobvanderlinden commented 8 years ago

Looks good, though I didn't really like dist residing in git. I haven't found a good way to work around this while still allowing easy inclusion in webpages. I've looked for ways we could use github's releases for CDN, but it seems not viable atm. So, dist on git it is ;)

machinekoder commented 8 years ago

I pretty much copied how it is done in the protobuf.js project https://github.com/dcodeIO/protobuf.js/tree/master/dist

A machintalk-dist repostory might be another solution which however adds complexity.