databricks-sdk-go

databricks-sdk-go copied to clipboard

If the path is equal to the root path, this error should bubble up to the caller. _Originally posted by @pietern in https://github.com/databricks/databricks-sdk-go/pull/208#discussion_r1034683108_

nfx

The default could be something like "empty response", as the current status is already included in the status message. The error below will become `failed to reach , got :...

nfx

For those known and mapped cases of required fields, generate some error messages to un-confuse users

nfx

nfx

Some string and complex field examples are in the OpenAPI. Comments should get unified example rendering.

nfx

nfx

For example: ``` // Skip this test if "Files in Repos" is not enabled. conf, err := wsc.WorkspaceConf.GetStatus(ctx, workspaceconf.GetStatusRequest{ Keys: "enableWorkspaceFilesystem", }) require.NoError(t, err) if (*conf)["enableWorkspaceFilesystem"] == "false" { ```...

pietern

Libraries API doesn't throw 404 on cluster autodeleted/not-found. See more details: https://github.com/databricks/terraform-provider-databricks/issues/1737

nfx

For example: http://localhost:6060/pkg/github.com/databricks/databricks-sdk-go/service/billing/#OutputFormat This contains enumeration, links, and code segments. Also see https://pkg.go.dev/github.com/fluhus/godoc-tricks. 

pietern

`WithZZZ` makes me think of `context.WithValue`, where you take something and return it with augmentation. This function sets a global, so should reflect that, IMO.

pietern