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 #20228: Document only ncf_const variables #863
Fixes #20228: Document only ncf_const variables #863
Conversation
Fixes #20228: Document only ncf_const variables
PR updated with a new commit |
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 don't see what would define these variables
@@ -541,22 +541,22 @@ There are also more variables available, all documented in https://docs.cfengine | |||
|Name | |||
|Description | |||
|
|||
|`const.dollar` | |||
|`rudder_const.dollar` |
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 don't have such on variable on my rudder 6.2
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 should states that these variables appears in 6.2.11 (or better, in 7.0) otherwise people with older version of rudder will have a broken config
PR was rebased on correct branch
https://issues.rudder.io/issues/20228