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
) is already using named arguments as well. Is it possible to update the current approach to make use of the named arguments for maintainability? It's much easier for someone to trace the code if we use named arguments. Positional is good with 1 or 2 arguments, not more than that.
The text was updated successfully, but these errors were encountered:
I noticed in this file
telescope/steps/engine/clusterloader2/slo/execute.yml
Line 22 in 3e0c127
We are not using named arguments such as --{somearg}={some value}. The current approach will eventually become very tough to maintain. The slo.py (
telescope/modules/python/clusterloader2/slo/slo.py
Line 227 in 3e0c127
The text was updated successfully, but these errors were encountered: