steve/databricks-cli
steve
/
databricks-cli
mirror of https://github.com/databricks/cli.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Do not prefill https:// in prompt for Databricks Host (#1364) 

## Changes
This PR is a minor UX improvement. By not autofilling the https://
prefix in Databricks Host we allow users to directly copy-paste from
their browser.

UX:
```
➜  cli git:(fix/copy-host) cli auth login
Databricks Profile Name: my-profile
Databricks Host (e.g. https://<databricks-instance>.cloud.databricks.com): https://foobar.cloud.databricks.com
Profile my-profile was successfully saved
```

## Tests
Manually.
This commit is contained in:
shreyas-goenka 2024-04-15 23:01:00 +05:30 committed by GitHub
parent 1f1fe4c6a8
commit b71f853649
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
1 changed files with 1 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
cmd/auth/auth.go
View File

@ -35,9 +35,7 @@ GCP: https://docs.gcp.databricks.com/en/dev-tools/auth/index.html`,
func promptForHost(ctx context.Context) (string, error) { func promptForHost(ctx context.Context) (string, error) {
prompt := cmdio.Prompt(ctx) prompt := cmdio.Prompt(ctx)
prompt.Label = "Databricks Host" prompt.Label = "Databricks Host (e.g. https://<databricks-instance>.cloud.databricks.com)"
prompt.Default = "https://"
prompt.AllowEdit = true
// Validate? // Validate?
host, err := prompt.Run() host, err := prompt.Run()
if err != nil { if err != nil {