mirror of https://github.com/databricks/cli.git
068c7cfc2d
## Changes With https://github.com/databricks/cli/pull/1507 and https://github.com/databricks/cli/pull/1511 we are clarifying the semantics associated with `dyn.InvalidValue` and `dyn.NilValue`. An invalid value is the default zero value and is used to signals the complete absence of the value. A nil value, on the other hand, is a valid value for a piece of configuration and signals explicitly setting a key to nil in the configuration tree. In keeping with that theme, this PR returns `dyn.InvalidValue` instead of `dyn.NilValue` at error sites. This change is not expected to have a material change in behaviour and is being done to set the right convention since we have well-defined semantics associated with both `NilValue` and `InvalidValue`. ## Tests Unit tests and integration tests pass. Also manually scanned the changes and the associated call sites to verify the `NilValue` value itself was not being relied upon. |
||
|---|---|---|
| .. | ||
| convert | ||
| dynassert | ||
| dynvar | ||
| merge | ||
| yamlloader | ||
| yamlsaver | ||
| kind.go | ||
| kind_test.go | ||
| location.go | ||
| location_test.go | ||
| mapping.go | ||
| mapping_test.go | ||
| path.go | ||
| path_string.go | ||
| path_string_test.go | ||
| path_test.go | ||
| pattern.go | ||
| pattern_test.go | ||
| value.go | ||
| value_test.go | ||
| value_underlying.go | ||
| value_underlying_test.go | ||
| visit.go | ||
| visit_get.go | ||
| visit_get_test.go | ||
| visit_map.go | ||
| visit_map_test.go | ||
| visit_set.go | ||
| visit_set_test.go | ||
| walk.go | ||
| walk_test.go | ||