search

databricks / cli

Databricks CLI
Other
132 stars 50 forks source link

Remove `paths.Paths` from bundle resources #1668

Closed shreyas-goenka closed 1 month ago

shreyas-goenka commented 1 month ago

Changes

Traditionally we used paths.Paths to track the file path associated with a bundle resource. Underneath the hood this struct was already using the location associated with the dynamic value. This PR removes this unnecessary indirection and changes callsites to directly fetch the file path from the dynamic value when they need it.

Tests

Existing tests.

shreyas-goenka commented 1 month ago

Triggered integration tests...

shreyas-goenka commented 1 month ago

After another look, I realized we do not need to track the location at the resource level at all. Removing it all together in this PR: https://github.com/databricks/cli/pull/1673/files