This repository has been archived by the owner on Feb 12, 2024. It is now read-only.

node-multistream #24

Closed

daviddias opened this issue Sep 15, 2015 · 1 comment

Member

daviddias commented Sep 15, 2015

node-multistream

https://github.com/diasdavid/node-multistream

Follows https://github.com/jbenet/multistream/tree/master/multistream spec

How you can help

make code coverage go up to 100%

daviddias added help wanted labels

RichardLitt changed the title ~~# node-multistream~~ node-multistream

Member Author

daviddias commented Dec 11, 2015

-> libp2p/js-libp2p#5

daviddias closed this as completed

MicrowaveDev pushed a commit to galtproject/js-ipfs that referenced this issue


          fix: number is not a valid mtime value (ipfs#24)

bb2d841

* fix: number is not a valid mtime value

See https://github.com/ipfs/js-ipfs-unixfs#create-an-unixfs-data-element  - it should be `Date | object | array`.

Ends up ignored https://github.com/ipfs/js-ipfs-unixfs/blob/master/src/index.js#L104-L106

* refactor: use stat.mtime, it is already a Date

vasco-santos added a commit that referenced this issue


          chore: update ipfs dep (#24)

47ccbe1

# for free to subscribe to this conversation on GitHub. Already have an account? #.