Skip to content

apps: add git flags to create/update/deploy - #6182

Open
atreyadbrx wants to merge 1 commit into
databricks:mainfrom
atreyadbrx:add-apps-git-flags
Open

apps: add git flags to create/update/deploy#6182
atreyadbrx wants to merge 1 commit into
databricks:mainfrom
atreyadbrx:add-apps-git-flags

Conversation

@atreyadbrx

Copy link
Copy Markdown

The apps create/update/deploy commands accept a git repository and git deployment source, but the code generator emits these nested objects as // TODO: complex arg so they were only reachable via --json. Add ergonomic top-level flags for the GA git fields:

  • create/update: --git-url, --git-provider (App.GitRepository)
  • deploy: --git-branch, --git-tag, --git-commit, --git-source-code-path (AppDeployment.GitSource)

The nested SDK pointers stay nil unless a git flag is set, so non-git requests are unchanged. Validation matches the API contract: url and provider must be set together; branch/tag/commit are mutually exclusive; source-code-path requires a ref.

Co-authored-by: Isaac

Changes

Why

Tests

The apps create/update/deploy commands accept a git repository and git
deployment source, but the code generator emits these nested objects as
`// TODO: complex arg` so they were only reachable via --json. Add
ergonomic top-level flags for the GA git fields:

- create/update: --git-url, --git-provider (App.GitRepository)
- deploy: --git-branch, --git-tag, --git-commit, --git-source-code-path
  (AppDeployment.GitSource)

The nested SDK pointers stay nil unless a git flag is set, so non-git
requests are unchanged. Validation matches the API contract: url and
provider must be set together; branch/tag/commit are mutually exclusive;
source-code-path requires a ref.

Co-authored-by: Isaac
@github-actions

github-actions Bot commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

An authorized user can trigger integration tests manually by following the instructions below:

Trigger:
go/deco-tests-run/cli

Inputs:

  • PR number: 6182
  • Commit SHA: 43148e8b766de7ac0497aa87e229a57374e01870

Checks will be approved automatically on success.

@github-actions

github-actions Bot commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

Waiting for approval

Based on git history, these people are best suited to review:

  • @pietern -- recent work in cmd/workspace/apps/

Eligible reviewers: @MarioCadenas, @Shridhad, @atilafassina, @calvarjorge, @ditadi, @fjakobs, @igrekun, @keugenek, @pffigueiredo, @pkosiec

Suggestions based on git history. See OWNERS for ownership rules.

Sign up for free to 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.

1 participant