Results 59 issues of Pieter Noordhuis
trafficstars

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

enhancement

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" { ```...

enhancement
discussion

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. ![Screenshot 2022-11-23 at 09 18 09](https://user-images.githubusercontent.com/9845/203499283-0ea36676-681a-401d-ac8f-08be8fd63c2f.png)

enhancement

`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.

discussion

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