-
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
fix: fix non-disposable and normalises behaviour #379
Conversation
f95d011
to
5c1a6ce
Compare
package.json
Outdated
"execa": "^2.0.4", | ||
"fs-extra": "^8.1.0", | ||
"hat": "~0.0.3", | ||
"ipfs-http-client": "^36.0.0", | ||
"ipfs-http-client": "^36.1.0", |
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.
v37
is out - breaking change is minor, just downgrades multiaddr
from v7
to v6
until we can upgrade all js-ipfs*
in one go.
const daemonOptions = merge({ | ||
exec: this.options.exec, | ||
type: this.options.type, | ||
IpfsClient: this.options.IpfsClient, |
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.
I wish this was just client
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.
Not actionable feedback, please ignore.
This PR reworks daemon spawn options and handling of start/init, it also makes sure spawned daemons attach themselves to running daemons on the default/provided repo.
Some dependencies were also removed.
Ref: #304
closes #305
closes #276
closes #354
closes #330
closes #329