search

databricks / cli

Databricks CLI
Other
115 stars 39 forks source link

merge.Override: Fix handling of dyn.NilValue #1530

Closed kanterov closed 4 days ago

kanterov commented 5 days ago

Changes

Fix handling of dyn.NilValue in merge.Override in case dyn.Value has location

Tests

Unit tests

codecov-commenter commented 5 days ago

Codecov Report

Attention: Patch coverage is 66.66667% with 1 line in your changes missing coverage. Please review.

Project coverage is 53.91%. Comparing base (e22dd8a) to head (9364bde). Report is 168 commits behind head on main.

Files Patch % Lines
libs/dyn/merge/override.go 66.66% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #1530 +/- ## ========================================== + Coverage 52.25% 53.91% +1.65% ========================================== Files 317 353 +36 Lines 18004 20647 +2643 ========================================== + Hits 9408 11131 +1723 - Misses 7903 8707 +804 - Partials 693 809 +116 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.