Skip to content

fix(cli): Support list alias for sentry cli defaults - #1539

Merged
BYK merged 1 commit into
mainfrom
seer/fix/cli-defaults-list-alias
Oct 17, 2026
Merged

BYK merged 1 commit into
mainfrom
seer/fix/cli-defaults-list-alias

Conversation

@sentry

@sentry sentry Bot commented Oct 17, 2026

Copy link
Copy Markdown
Contributor

Summary of changes

  • Support list, ls, and show as aliases to show all defaults in sentry cli defaults <key> [<value>] instead of failing with ValidationError: Unknown default 'list'.
  • Disallow passing extra values or --clear alongside the show-all verb aliases with an informative error message.
  • Update command documentation examples to note the list alias.
  • Add unit tests covering list, ls, show, LIST, value rejection, and --clear rejection on verb aliases.

Fixes CLI-1CT

@sentry <feedback>: Autofix iterates on these changes
@sentry stop iterating: Autofix stops iterating on this run

This PR was automatically generated by Sentry. You can adjust this setting at any time.

@vercel

vercel Bot commented Oct 17, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Actions Updated
sentry-local Skipped Skipped Oct 17, 2026 10:43pm UTC

Request Review

@github-actions github-actions Bot added the risk: low PR risk score: low label Oct 17, 2026
@BYK
BYK merged commit d7aa182 into main Oct 17, 2026
26 checks passed
@BYK
BYK deleted the seer/fix/cli-defaults-list-alias branch October 17, 2026 23:23

This branch was previously deployed

1 inactive deployment
Preview — 8e0b1de5 Deployed Oct 17, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

risk: low PR risk score: low

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant