-
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
Fragmentation #136
Comments
Found this one too https://github.com/haadcode/ipfs-test-apis |
@haadcode we need to converge https://github.com/haadcode/js-ipfsd-ctl into this. What are the steps for that? What is stopping you from using the main one and maintaining a fork? |
Short: 0.4.5. Long: So, to unravel all that, we need to merge ipfs-inactive/npm-go-ipfs-dep#18 and ipfs-inactive/npm-go-ipfs-dep#19 and release The other part is to merge #145 and release new version of That's all! :) Finally there 🎉 🎆 |
I see a breaking change proposed on #145 and no PRs to js-ipfs and js-ipfs-api that cope with that change, meaning that the poor soul that merges and releases would have to run and put up fires. Can we avoid this by preemptively having PRs that cope with the fixes so that no one is caught off guard? |
All done \o/ |
TIL that there is a @haad/ipfsd-ctl. I don't understand why there is a need to create such a module, but it seems it has been going for a while, making it more than an experiment, but an actual fork/module available in NPM.
I very much disagree with this approach because it is clearly fragmenting the ecosystem set by creating yet another module that is not being tracked as a PR and doesn't show any future where that will happen, making us maintain 2 modules instead of one.
When the challenge on ipfs/js-ipfs#556 was launched, the proposal was exactly to reduce fragmentation and to make the launch of IPFS nodes more easily, but what I see that got created was another module, ipfs-daemon, which has its set of issues, which uses a module that is a fork of this one, namely @haad/ipfsd-ctl.
@haadcode can we converge all of these, it really doesn't help. Others are invited to provide their feedback, I might also be missing the value and apologize in advance if I'm doing so.
The text was updated successfully, but these errors were encountered: