Make stream definitions consistent

bergos commented 5 years ago

This PR should fix #2.

Two remarks:

In Typical use cases we could add more types, but maybe that would be to confusing in the end. Stream stream could be Stream<Buffer or String or Object> stream. Buffer to cover binary formats and encoding specific text formats, String for text formats and Object for formats which are serialized into JS objects, like JSON-LD.
Stream extends from EventEmitter, but that interface is not defined in the spec. Do we want to reference to the Node.js EventEmitter class or define our own interface in the spec? Could be a topic for a different PR.

elf-pavlik commented 5 years ago

@blake-regalia @RubenVerborgh I think we should merge it ASAP, last chance to review it!

RubenVerborgh commented 5 years ago

No objections from my side 👍

rdfjs / stream-spec