brunch / auto-reload-brunch

Adds automatic browser reloading support to brunch.
88 stars 30 forks source link

Use node.js web socket server to exclude c deps #40

Closed paulmillr closed 9 years ago

paulmillr commented 10 years ago

https://gist.github.com/bradleywright/1021082

paulmillr commented 10 years ago

we may want to create new lib for this

es128 commented 10 years ago

Have you tried out that gist? The c deps have never bothered me at all, node/npm makes it really seamless.

I'm reluctant to abandon ws, it seems really solid (I use it elsewhere). I know the stakes here are very low with this, but I'm worried about opening a floodgate of bugs and edge cases with an unproven solution.

paulmillr commented 10 years ago

i hadn't tried this, but I think we should at least try