-
Notifications
You must be signed in to change notification settings - Fork 73
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 #13615: Bad explanation for generic variable overide priority cause people making the opposite of what is needed #2042
Conversation
<p>Unique Directives can be applied only once (ie. Time Settings), so only the highest priority will be appllied.</p> | ||
<p>Priority has two uses depending if the technique from which that directive is derived is <b>Unique</b> or not.</p> | ||
<p>Unique directives can be applied only once (for example Time Settings), so only the highest priority will be applied.</p> | ||
<p>For <b>non-unique</b> directives, priority is used to choose the order of the directive linearisation when applicable. The |
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 think linearisation is clear for users. Maybe application?
PR rebased |
993b6aa
to
789c032
Compare
<p>Priority has two uses depending if the technique from which that directive is derived is <b>Unique</b> or not.</p> | ||
<p>Unique directives can be applied only once (for example Time Settings), so only the highest priority will be applied.</p> | ||
<p>For <b>non-unique</b> directives, priority is used to choose the order of the directive application when applicable. The | ||
highest priority directive comes first. Note that overriding variables definitions are the ones coming last.</p> |
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.
Note that overriding variables definitions are the ones coming last.
.. so the used value will be the one with the lowest priority
…ause people making the opposite of what is needed
PR rebased |
789c032
to
4940c73
Compare
OK, merging this PR |
https://www.rudder-project.org/redmine/issues/13615