databricks / cli

Databricks CLI
Other
148 stars 56 forks source link

Add `dyn.Time` to box a timestamp with its original string value #1732

Closed pietern closed 2 months ago

pietern commented 2 months ago

Changes

If not explicitly quoted, the YAML loader interprets a value like 2024-08-29 as a timestamp. Such a value is usually intended to be a string instead. Our normalization logic was not able to turn a time value back into the original string.

This change boxes the time value to include its original string representation. Normalization of one of these values into a string can now use the original input value.

Tests

Unit tests in libs/dyn/convert.