-
Notifications
You must be signed in to change notification settings - Fork 62
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[WIP] feat: add ability to start js-ipfs nodes #170
Conversation
package.json
Outdated
"release": "aegir release -t node", | ||
"release-minor": "aegir release --type minor -t node", | ||
"release-major": "aegir release --type major -t node", | ||
"coverage-publish": "aegir coverage -u" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please make an aegir upgrade be part of a separate PR
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
kk, will pull aegir out.
4d814b1
to
ec8de2a
Compare
this depends on ipfs/js-ipfs#1095 now. |
test/spawning-daemons.js
Outdated
// fails on windows see https://github.com/ipfs/js-ipfs-api/issues/408 | ||
if (isWindows) { | ||
return it.skip('uses the correct ipfs-api') | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This skip has been removed in 381ac1c, you may have to rebase https://github.com/ipfs/js-ipfsd-ctl/network
ec8de2a
to
e0f7efd
Compare
e0f7efd
to
593c534
Compare
Is this still needed with #176 ? |
this is not needed anymore - closing. |
This depends on - ipfs/js-ipfs#1067