-
Notifications
You must be signed in to change notification settings - Fork 3.2k
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
docs: mark generated python SDK as deprecated #13719
Conversation
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.
sdks/python/README.md
should also be updated
There are already two good alternatives for using python to create workflows
Couler does not seem to be actively maintained, but yes Hera ofc is
and probably others
yea there's a small mountain of bugs with the SDKs; some of which are resolved in updating the OpenAPI generator, which is itself a breaking change 🙃
Signed-off-by: Alan Clucas <alan@clucas.org>
Signed-off-by: Alan Clucas <alan@clucas.org>
bc33dfb
to
a9ee8b5
Compare
Eugh, I don't know how I missed this one. Thanks.
I've reordered the list and pushed for hera being the suggestion a bit more heavily. |
Co-authored-by: Anton Gilgur <4970083+agilgur5@users.noreply.github.com> Signed-off-by: Isitha Subasinghe <isitha@pipekit.io>
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
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.
Since Isitha applied my suggestion above, this LGTM now
Fixes #13703
Motivation
The Python SDK is auto-generated, and the maintainers of argo-workflows do not wish to invest their limited time improving this situation. There are already two good alternatives for using python to create workflows
This will allow us to close
and probably others
Modifications
The main change is to mark the SDK as deprecated in client-libraries.md, and give a planned removal in 3.7.
There should be a follow up once
release/3.6
is branched to actually remove the generation of the code.Verification
make docs
passes