databricks-sdk-py
databricks-sdk-py copied to clipboard
Added `w.config.account_host` to get the relevant account host from a workspace client
Changes
This simplifies downstream integration testing as well as configuration
Tests
- [ ]
make test
run locally - [ ]
make fmt
applied - [ ] relevant integration tests applied
Codecov Report
All modified lines are covered by tests :white_check_mark:
Files | Coverage Δ | |
---|---|---|
databricks/sdk/azure.py | 100.00% <100.00%> (ø) |
|
databricks/sdk/core.py | 80.69% <100.00%> (+0.32%) |
:arrow_up: |
:loudspeaker: Thoughts on this report? Let us know!.
One nit to make this work for dev/staging & label as experimental, but otherwise this seems reasonable to me if it unblocks you/customers.
all the info here is fine to be public ( and already public)
this needs a bit more adjustments to work
Are similar PRs for the other SDKs tracked some place?
@pietern i'll have one for GoLang the following week - once i figure out the way to have it friendly for unit tests, need the same thing for two projects