-
Notifications
You must be signed in to change notification settings - Fork 1
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
Rename get_transformation_list
to list_transformations
#46
Comments
Shouldn't this better go in the standard app API, since we want to rename the capability? I do agree, though. It is also not consistent with other like the semantic mappings:
|
I don't agree that the function names in the SDK and the operationIDs (capabilities) must correspond one-to-one. One is a RESTful API, the other one is a Python library so we should use conventions that are applicable in the respective context. |
I meant this because the capabilities also reflect this discrepancy. There is |
Ah yes, that should probably be renamed as well. |
I propose to rename the
get_transformation_list()
function tolist_transformations()
for conciseness:python-sdk/marketplace/app/v0/transformation.py
Line 9 in 32e2064
The text was updated successfully, but these errors were encountered: