Skip to content

remove/cli: drop unused validate commands#1310

Merged
burmudar merged 1 commit intomainfrom
wb/remove-validate-cmd
May 4, 2026
Merged

remove/cli: drop unused validate commands#1310
burmudar merged 1 commit intomainfrom
wb/remove-validate-cmd

Conversation

@burmudar
Copy link
Copy Markdown
Contributor

@burmudar burmudar commented May 4, 2026

The legacy src validate entrypoint is unused, so keeping it around adds dead code and extra maintenance surface in the CLI. This removes the top-level src validate command and its install and kube subcommands, which were still wired through the older command registration path. For more context, see https://sourcegraph.slack.com/archives/C022SPMNR0W/p1777905299691889

Test Plan

  • go test ./cmd/src/...

The legacy `src validate` entrypoint is unused, so keeping it around adds dead code and extra maintenance surface in the CLI. This removes the top-level `src validate` command and its `install` and `kube` subcommands, which were still wired through the older command registration path. For more context, see https://sourcegraph.slack.com/archives/C022SPMNR0W/p1777905299691889

## Test Plan

- go test ./cmd/src/...
@burmudar burmudar self-assigned this May 4, 2026
@burmudar burmudar requested a review from a team May 4, 2026 15:22
@burmudar burmudar enabled auto-merge (squash) May 4, 2026 15:23
@burmudar burmudar merged commit 284f08b into main May 4, 2026
9 checks passed
@burmudar burmudar deleted the wb/remove-validate-cmd branch May 4, 2026 15:28
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.

2 participants