You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
In function.py we have 2 different execute functions, and it's really difficult when calling for instance function.execute([to know which one is used, we have to track the function object type.
I propose to rename one of them, like execute_function for the one in wrapper or execute_cli for the one running the cli
The text was updated successfully, but these errors were encountered:
I agree that it is confusing. I propose to keep execute for the cli as it is the function exposed to the user, in order to keep a name as simple as possible, and to rename to execute_function for the wrapper.
In function.py we have 2 different
execute
functions, and it's really difficult when calling for instancefunction.execute([
to know which one is used, we have to track thefunction
object type.I propose to rename one of them, like
execute_function
for the one in wrapper orexecute_cli
for the one running the cliThe text was updated successfully, but these errors were encountered: