-
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 #9230: Define a dry-run lib when default bodies are not supported #467
Conversation
This requires changes in |
Commit modified |
9c36d80
to
81dceda
Compare
@@ -34,6 +54,11 @@ | |||
# WARNING: * promise type specific dry-run is NOT considered by action bodies, and will be overriden | |||
# WARNING: when using any action body. | |||
# WARNING: | |||
# | |||
# @parameter mode The dry-run mode, "true" to enable dry-run-mode and "false" to disable it | |||
# @agent_requirements "capabilities": [ "defaultbody" ] || "agent_version"" >= 3.9 |
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.
This kind of expression is not yet supported
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.
I know, but we will need it for this. We need to choose the syntax and implement it in list-compatible-inputs.
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.
It was not implemented because it is very long and complex to implement in standard shell, I hope we can find time to implement it before the end of times
81dceda
to
c4f1f63
Compare
Commit modified |
OK, merging this PR |
https://www.rudder-project.org/redmine/issues/9230