Skip to content

apps init: show template ref in header and clone spinner - #5249

Merged
calvarjorge merged 1 commit into
mainfrom
jorge.calvar/apps_init_template_version_refactor
May 13, 2026
Merged

apps init: show template ref in header and clone spinner#5249
calvarjorge merged 1 commit into
mainfrom
jorge.calvar/apps_init_template_version_refactor

Conversation

@calvarjorge

@calvarjorgecalvarjorge commented May 13, 2026

Copy link
Copy Markdown
Contributor

Summary

Removes the standalone Using AppKit template version X log line from databricks apps init. The resolved ref now appears:

  • On the interactive header (before the project-name prompt) as a dimmed Template version 0.24.0 line, so the user can cancel before committing to a name.
  • On the clone spinner and done line as a suffix, e.g. Cloning template (version 0.24.0)... / ✔ Cloning template (version 0.24.0).

The label tracks the source of the ref:

InvocationLabel
auto-resolved (default) / --version Xversion X
--branch X (default or custom template)branch X
--template URL with no branch(no label)

On a fallback to the embedded AppKit version, the spinner reflects the version actually cloned.

Before
image

After
image
image

Test plan

  • dbs apps init --name foo — version appears on the spinner, not as a separate line
  • dbs apps init (interactive) — header shows the version before the name prompt; cancel before naming works
  • dbs apps init --version 0.23.0 --name foo — spinner shows (version 0.23.0)
  • dbs apps init --branch some-branch --name foo — spinner shows (branch some-branch)
  • dbs apps init --template <URL> --name foo — no ref label (current behavior preserved)

Drops the standalone "Using AppKit template version X" line. The
resolved ref (e.g. "version 0.24.0" or "branch feature-x") now appears
on the interactive header before the name prompt and as a suffix on
the clone spinner so the user can cancel before naming the project.
@calvarjorge
calvarjorgeforce-pushed the jorge.calvar/apps_init_template_version_refactor branch from f54d331 to e83e442CompareMay 13, 2026 12:46
@calvarjorge
calvarjorge requested a review from pkosiecMay 13, 2026 12:50

@pkosiecpkosiec left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@calvarjorge
calvarjorge enabled auto-merge May 13, 2026 13:24
@calvarjorge
calvarjorge added this pull request to the merge queueMay 13, 2026
Merged via the queue into main with commit f213ed0May 13, 2026
27 checks passed
@calvarjorge
calvarjorge deleted the jorge.calvar/apps_init_template_version_refactor branch May 13, 2026 13:40
denik pushed a commit that referenced this pull request May 20, 2026
## Summary
Removes the standalone `Using AppKit template version X` log line from
`databricks apps init`. The resolved ref now appears:
- On the interactive header (before the project-name prompt) as a dimmed
`Template version 0.24.0` line, so the user can cancel before committing
to a name.
- On the clone spinner and done line as a suffix, e.g. `Cloning template
(version 0.24.0)...` / `✔ Cloning template (version 0.24.0)`.
The label tracks the source of the ref:
| Invocation | Label |
|---|---|
| auto-resolved (default) / `--version X` | `version X` |
| `--branch X` (default or custom template) | `branch X` |
| `--template URL` with no branch | (no label) |
On a fallback to the embedded AppKit version, the spinner reflects the
version actually cloned.
**Before**
<img width="409" height="102" alt="image"
src="https://github.com/user-attachments/assets/910e3685-93e7-4adb-9afd-9d547ca4d64b"
/>
**After**
<img width="306" height="56" alt="image"
src="https://github.com/user-attachments/assets/05755405-2874-4ad3-899d-0f7ad405f71c"
/>
<img width="436" height="162" alt="image"
src="https://github.com/user-attachments/assets/23d25c7d-25fd-4c83-9043-6f2c1004ebe6"
/>
## Test plan
- [x] `dbs apps init --name foo` — version appears on the spinner, not
as a separate line
- [x] `dbs apps init` (interactive) — header shows the version before
the name prompt; cancel before naming works
- [x] `dbs apps init --version 0.23.0 --name foo` — spinner shows
`(version 0.23.0)`
- [x] `dbs apps init --branch some-branch --name foo` — spinner shows
`(branch some-branch)`
- [x] `dbs apps init --template <URL> --name foo` — no ref label
(current behavior preserved)
TanishqDatabricks pushed a commit to TanishqDatabricks/cli that referenced this pull request May 22, 2026
…5249)
## Summary
Removes the standalone `Using AppKit template version X` log line from
`databricks apps init`. The resolved ref now appears:
- On the interactive header (before the project-name prompt) as a dimmed
`Template version 0.24.0` line, so the user can cancel before committing
to a name.
- On the clone spinner and done line as a suffix, e.g. `Cloning template
(version 0.24.0)...` / `✔ Cloning template (version 0.24.0)`.
The label tracks the source of the ref:
| Invocation | Label |
|---|---|
| auto-resolved (default) / `--version X` | `version X` |
| `--branch X` (default or custom template) | `branch X` |
| `--template URL` with no branch | (no label) |
On a fallback to the embedded AppKit version, the spinner reflects the
version actually cloned.
**Before**
<img width="409" height="102" alt="image"
src="https://github.com/user-attachments/assets/910e3685-93e7-4adb-9afd-9d547ca4d64b"
/>
**After**
<img width="306" height="56" alt="image"
src="https://github.com/user-attachments/assets/05755405-2874-4ad3-899d-0f7ad405f71c"
/>
<img width="436" height="162" alt="image"
src="https://github.com/user-attachments/assets/23d25c7d-25fd-4c83-9043-6f2c1004ebe6"
/>
## Test plan
- [x] `dbs apps init --name foo` — version appears on the spinner, not
as a separate line
- [x] `dbs apps init` (interactive) — header shows the version before
the name prompt; cancel before naming works
- [x] `dbs apps init --version 0.23.0 --name foo` — spinner shows
`(version 0.23.0)`
- [x] `dbs apps init --branch some-branch --name foo` — spinner shows
`(branch some-branch)`
- [x] `dbs apps init --template <URL> --name foo` — no ref label
(current behavior preserved)
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@calvarjorge@pkosiec@MarioCadenas