-
Notifications
You must be signed in to change notification settings - Fork 30
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
release 0.4.x #10
Comments
wait, why isn't this happening!? it's part of the go-ipfs release checklist https://github.com/ipfs/go-ipfs/blob/master/docs/releases.md cc @whyrusleeping @Kubuxu ?? |
It appears this doesnt happen because this package relies on gobuilder |
@diasdavid mentioned there was a different npm package for getting the go-ipfs daemon in javascript. |
js-ipfsd-ctl relies on For now it is just 0.4.1, as we can't update to 0.4.2 due to a bug in 0.4.2. @JGAntunes is doing a really awesome work of upgrading js-ipfsd-ctl to use binwrapper, that will solve the Arch problem ipfs/js-ipfsd-ctl#89 I'll go ahead an update this module description to point to go-ipfs-dep. |
@diasdavid if I read this correctly, I understand that:
Correct? |
Looks as if this is still not resolved. If this module is deprecated then it needs a deprecate message adding in npm to point to the correct place. Also the readme there needs updating to install the correct npm package, and not this one. |
guys, come on! people depend on this module-- including me. it wasted 20min of my time to realize that this was go-ipfs 0.3x being installed... |
this module should not be deprecated-- why do people think it is? the goal of this module is to enable this: > npm install go-ipfs
> node_modules/.bin/ipfs version and this > npm install -g go-ipfs
> ipfs version as the readme says, to install go-ipfs. it should be using dist.ipfs.io |
I'm on this, fixing it. |
Okay, fixed. I published:
sorry for the silly version suffixes.
|
Awesome, thanks. For the record, this is exactly what I want too: an executable that can be run from the shell, but managed via NPM. |
Not sure where to properly file this, but maybe there could be a check on CI that would fail if all of the steps for a given release were not completed (at least those, like this one, that are easy to check automatically). |
No description provided.
The text was updated successfully, but these errors were encountered: