Skip to content
This repository has been archived by the owner on Mar 10, 2020. It is now read-only.

0.3.10 #169

Merged
merged 6 commits into from
Dec 19, 2015
Merged

0.3.10 #169

merged 6 commits into from
Dec 19, 2015

Conversation

daviddias
Copy link
Contributor

I've started this, found a bug, but seems to be on the gulp karma task and not js-ipfs-api itself, tests print a ton more stuff then they used too.

Running the tests on master yield the same error, @dignifiedquire any ideas? My first thought is that there was some patch upgrade in some of the babel+gulp+karma dependencies that broke things, which makes sense why all greenkeeper PRs now warn of breaking build

moved to gist

@dignifiedquire
Copy link
Contributor

@diasdavid looks like webpack logging needs be silenced

@dignifiedquire
Copy link
Contributor

  •  Tests passing locally
  •  Use ndjson to parse streamed output

@dignifiedquire dignifiedquire changed the title WIP: 0.3.10 0.3.10 Dec 18, 2015
@dignifiedquire
Copy link
Contributor

@diasdavid as far as I'm concerned this is ready to merge

@daviddias
Copy link
Contributor Author

awesome 👍 will look into this tomorrow properly to merge and publish :)

daviddias added a commit to daviddias/vinyl-fs that referenced this pull request Dec 19, 2015
We have hit a issue on ipfs-inactive/js-ipfs-http-client#169 because glob-stream 5.3.0 has a new version of micromatch which breaks js-ipfs-api. (please someone call the semver police 🚨 :P)

@dignifiedquire has locked 5.2.0 on js-ipfs-api, but to avoid having to cause more people pass through the debugging of this problem if they decide to use this module, I'm just going to make it more strict here too.
@daviddias
Copy link
Contributor Author

Thank you @dignifiedquire

@fazo96 I've also submitted a small PR to jbenet/vinyl-fs#3 so that in the future other people that use it don't hit the same problem

daviddias added a commit that referenced this pull request Dec 19, 2015
@daviddias daviddias merged commit 30e9ab6 into master Dec 19, 2015
@daviddias daviddias deleted the 0.3.10 branch December 19, 2015 14:46
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants