Pieter Noordhuis
Pieter Noordhuis
For consistency with other upcoming SDKs.
Couple validations come to mind: 1. Strings must be alphanumerical 2. Cannot use names already in use by other fields
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" { ```...
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. 
`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.
Note: this requires https://github.com/databricks/cli/pull/1219.
## Changes Running this action before doing an actual release should prevent situations like #3645 where it doesn't work on release. ## Tests See workflows.
## Changes **Note: this does not work yet. We need to prefix workspace paths with `/Workspace` first.** Fixes #1517. ## Tests tbd