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
refactor: remove embedded ipfs #3969
base: main
Are you sure you want to change the base?
Conversation
- we have a heap of tests that still use the "legacy" models and said tests are tightly coupled to IPFS. I am going to delete these tests in a follow on commit to this.
Important Auto Review SkippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (invoked as PR comments)
Additionally, you can add CodeRabbit Configration File (
|
I am sorry but I don't agree with the approach. You are removing almost all of our test scenarios, and pushing a commit with no support for IPFS users that can block our release pipeline until it is added back in. We still need to give notice to our users and inform them to spin up their own IPFS node instead of the embedded one Instead of deleting the tests, the right approach is to migrate them to the new models as planned in #3832, and to local publisher instead of IPFS as planned in #3816. Now we can speed things and remove embedded ipfs from tests before 1.5 release, but we shouldn't do more intrusive things or fully remove embedded ipfs without communicating that early to users. |
@wdbaruni agreed, which is why this remains in draft. |
You've asked me to review this proposal and I did :) |
model
) we plan to remove very soon as we move to the new API andmodels
package. We can always add similar testing back in a follow on.