search

databricks / cli

Databricks CLI
Other
132 stars 50 forks source link

Introduce new type for a slice of dyn.Location #1611

Closed shreyas-goenka closed 1 month ago

shreyas-goenka commented 2 months ago

Changes

Small quality of life improvement, since we now have multiple locations everywhere in the code.

shreyas-goenka commented 2 months ago

I'll resume work on this PR once https://github.com/databricks/cli/pull/1610 is merged.

shreyas-goenka commented 1 month ago

Meh, not worth it. This PR does not solve any problems so closing it.

pietern commented 1 month ago

FWIW, if we would do this it would be good to do the same for paths for symmetry (we use both in diags).