Browser support! #10

Closed
mcollina opened this Issue May 1, 2013 · 6 comments

Projects

None yet

3 participants

@mcollina
Owner

LevelGraph should run as is in the Browser, on top of level-js.
Some browserify-fu is needed.

@refset
Collaborator

I just played around with this and AFAICT the blocker seems to be the absence of a browserify compatible streams2 implementation (though there could be one I've not found?).

The other option of course is to add v0.8 support & rewrite levelgraph to use through instead of streams2.

Can you tell me if that interpretation is right?

@mcollina
Owner

Your intepretation is totally right. Maybe some guys of the browserify crew can help giving directions. Maybe @juliangruber might help :).
I don't mind using through, btw.

You might also need a ES5 shim for some bad browser out there, as I am depending on forEach, map, filter and so on.

@juliangruber

You can just use https://github.com/isaacs/readable-stream for streams2 support.

@mcollina
Owner

Then it should not be hard. Thanks @juliangruber!

@jez0990 do you fancy trying implementing browserify support and submitting a pull-request?

You should also switch the tests to https://npmjs.org/package/level-test.

@refset
Collaborator

A new branch is being created for this work: #22

@mcollina
Owner

Closed in favore of #23

@mcollina mcollina closed this Jul 1, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment