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

breaking change: stats spec, spec repo, stream to value on files read #679

Merged
merged 4 commits into from
Jan 27, 2018

Conversation

hacdias
Copy link
Contributor

@hacdias hacdias commented Jan 25, 2018

Ref.: ipfs-inactive/interface-js-ipfs-core#208.

  • tests: STAT spec
  • fix(breaking change): REPO spec
  • feat(breaking change): FILES spec: stream to value on files read

@daviddias
Copy link
Contributor

interface-ipfs-core changes merged and released. Have fun! :)

@daviddias
Copy link
Contributor

Also, make sure to update js-ipfs ;)

@hacdias
Copy link
Contributor Author

hacdias commented Jan 25, 2018

@diasdavid I'll update the interface-ipfs-core version here to try it out.

@hacdias
Copy link
Contributor Author

hacdias commented Jan 25, 2018

@diasdavid the tests for STATS are passing ;)

@@ -68,7 +68,7 @@
"eslint-plugin-react": "^7.5.1",
"go-ipfs-dep": "^0.4.13",
"gulp": "^3.9.1",
"interface-ipfs-core": "~0.41.0",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Use ~

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done!

@hacdias
Copy link
Contributor Author

hacdias commented Jan 25, 2018

@diasdavid anything missing here?

@daviddias
Copy link
Contributor

CI is not happy. Mind checking what's up? Seems that a rebase is needed.

@hacdias hacdias changed the title tests: stat spec breaking change: stats spec, spec repo Jan 26, 2018
@hacdias hacdias changed the title breaking change: stats spec, spec repo breaking change: stats spec, spec repo, stream to value on files read Jan 26, 2018
README.md Outdated
- [repo](https://github.com/ipfs/interface-ipfs-core/tree/master/SPEC/REPO.md)
- [`ipfs.repo.gc([options, callback])`](https://github.com/ipfs/interface-ipfs-core/tree/master/SPEC/REPO.md#gc)
- [`ipfs.repo.stat([options, callback])`](https://github.com/ipfs/interface-ipfs-core/tree/master/SPEC/REPO.md#stat)
- [`ipfs.repo.version([callback])`](https://github.com/ipfs/interface-ipfs-core/tree/master/SPEC/REPO.md#version)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should be under the Node Management to match the structure in interface-ipfs-core

@hacdias hacdias force-pushed the stats-spec branch 5 times, most recently from 6b954ba to 686617f Compare January 26, 2018 08:44
@hacdias
Copy link
Contributor Author

hacdias commented Jan 26, 2018

Everything seems to pass here except that the node wasn't spawned in 6 seconds so it failed.

@hacdias
Copy link
Contributor Author

hacdias commented Jan 27, 2018

@diasdavid I'd like to see everything passing, but I'm getting a "JavaScript heap out of memory" error. What should I do?

@daviddias
Copy link
Contributor

The "out of memory" issue started with the ipfsd-ctl update. Essentially what is happening is that WebPack is trying to bundle too much code (more than one version of ipfs-api) and that is crashing the whole process. There is a way to increase the amount of memory allocated. But the good way to do it is to avoid creating the issue.

I'm going to merge this PR but will create a issue to track this.

@daviddias daviddias merged commit 118456e into master Jan 27, 2018
@daviddias daviddias deleted the stats-spec branch January 27, 2018 17:54
@ghost ghost removed the in progress label Jan 27, 2018
@hacdias
Copy link
Contributor Author

hacdias commented Jan 27, 2018

Thanks!

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.

2 participants