search

FluxML / FluxJS.jl

I heard you like compile times
Other
42 stars 8 forks source link

Convert flux.js to an ES6 module? #12

Open amellnik opened 6 years ago

amellnik commented 6 years ago

Would people be open to splitting the js portion into a new repository, converting it to ES6 and adding it to npm? This would simplify the process of using it with webpack, managing dependencies, etc.

MikeInnes commented 6 years ago

I'm not against, but while the code is not that stable my thinking was that it might be easier just to inline the fluxjs code wherever it's needed – then you just have something that won't break when an update is released.

Do you have any thoughts on how to keep the Julia and JS sides of the codebase in sync, as they'd presumably have to be in different repos? Again, may just not be an issue if we stabilise a bit.

amellnik commented 6 years ago

I'm not sure of the best way to keep them in sync, but will keep thinking about it. I think you're right that this wouldn't make sense to do before things stabilize.