-
-
Notifications
You must be signed in to change notification settings - Fork 3k
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
core/commands!: remove deprecated object APIs #10375
Conversation
a3c68bf
to
c552d08
Compare
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.
Lgtm, but it may be safer if we also keep commands for patching directories.
141bdd3
to
6529303
Compare
This allows us to return meaningful error to end users who were not aware of deprecation, and also produce useful removal notice at https://docs.ipfs.tech/reference/kubo/rpc/ for people who try to execute commands or RPC from old tutorials.
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.
@hacdias thanks! lgtm, added some graceful errors – details inline.
if no concerns, feel free to merge
The idea is to remove all deprecated Object API commands (deprecated since 2021, #7936), except for
object diff
, which does not have an alternative.Closes #8887
Closes #3359
Closes #7384
Closes #5058
Closes #1724