-
Notifications
You must be signed in to change notification settings - Fork 86
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
Revert "home brew" code to official Python client #242
Comments
What do you mean by "the official Python API"? |
https://github.com/argoproj-labs/argo-client-python is unmaintained and you cannot easily support multiple versions of Argo Workflows either. We used it for validation but later removed it because there were some false positives. |
Can we use the OpenAPI specs then? Those are kept up to date... right?
…On Mon, Oct 4, 2021 at 5:05 PM Yuan Tang ***@***.***> wrote:
https://github.com/argoproj-labs/argo-client-python is unmaintained and
you cannot easily support multiple versions of Argo Workflows either. We
used it for validation but later removed it because there were some false
positives.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#242 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABXRO6OLCWTVI35KPCJLJTDUFGYEHANCNFSM5FHHDZTA>
.
Triage notifications on the go with GitHub Mobile for iOS
<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675>
or Android
<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
|
Please hold off on using https://github.com/argoproj/argo-workflows/tree/master/sdks/python until it's published to PyPI. There are still additional work to do there. |
Summary
Change custom code to use the official Python API
Use Cases
It's difficult developing new features and onboarding new developers to the codebase, as the underlying structures don't follow the (well-documented) official Python API.
Message from the maintainers:
Impacted by this bug? Give it a 👍. We prioritize the issues with the most 👍.
The text was updated successfully, but these errors were encountered: