-
Notifications
You must be signed in to change notification settings - Fork 38
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
Fixes #7278: Add new class parameter to command_execution_* #253
Fixes #7278: Add new class parameter to command_execution_* #253
Conversation
PR updated |
25cbede
to
569f054
Compare
Original PR #234 |
@@ -368,6 +368,58 @@ body classes classes_generic_return_codes_two(x, y, kept_return_code, repaired_r | |||
"promise_kept_$(y)", "$(y)_kept", "$(y)_ok", "$(y)_not_repaired", "$(y)_reached" }; | |||
} | |||
|
|||
# Define x prefixes/suffixed with promises outcome, based on return codes | |||
body classes classes_generic_return_codes_two(x, y, kept_return_codes, repaired_return_codes) |
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.
the name isn't really helpful in why it differs from the previous method
I forsee a lot of errors for users in the names of the methods (s or no s, that is the question ...)
569f054
to
b1be381
Compare
PR updated |
} | ||
|
||
# Define x prefixes/suffixed with true/false outcome, based on return codes | ||
body classes classes_generic_return_boolean_two(x, y, boolean_prefix, true_return_codes, false_return_codes) |
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.
why not classes_generic_return_boolean_list_two ?
PR updated |
b1be381
to
5cf5e6a
Compare
…r_to_command_execution Fixes #7278: Add new class parameter to command_execution_*
https://www.rudder-project.org/redmine/issues/7278