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
DM-29255 Modify applying config overrides for flexibility #173
Conversation
90971f1
to
4a3d1bd
Compare
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.
Looks good, small bunch of minor comments/questions.
@@ -125,35 +227,77 @@ def applyTo(self, config): | |||
------ | |||
`Exception` is raised if operations on configuration object fail. | |||
""" | |||
# Look up a stack of variables people may be useing when setting |
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.
typo: useing
# The field might be a string corresponding to a attribute | ||
# hierarchy, attempt to split off the last field which | ||
# will then be set. | ||
parent, *child = field.rsplit(".", maxsplit=1) |
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.
rpartition
may be slightly more readable, but that is a personal preference
31b495a
to
3843557
Compare
These changes allow greater flexibility when doing config overrieds in areas such as not needing string escaping when working with the command line, and using values from python statements in other parts of pipeline config blocks.
3843557
to
9496650
Compare
These changes allow greater flexibility when doing config overrieds
in areas such as not needing string escaping when working with the
command line, and using values from python statements in other
parts of pipeline config blocks.