Skip to content

Add command header and dry-run flag#340

Merged
jdalton merged 7 commits intomainfrom
dry_header
Feb 26, 2025
Merged

Add command header and dry-run flag#340
jdalton merged 7 commits intomainfrom
dry_header

Conversation

@pvdz
Copy link
Contributor

@pvdz pvdz commented Feb 26, 2025

This PR

  • Adds a command header (currently disabled by default, you can make it appear with SOCKET_CLI_SHOW_BANNER=1)
  • Adds a --dry-run flag to all commands except cdxgen
  • Tests the --dry-run flag for all commands, asserting their output
  • Fixed a bunch of commands. Some had the wrong command in the config, some had inconsistent error handling, etc.
  • Exit code for invalid args/flags is now 2 for all commands
  • No longer auto-showing the help when input validation fails

@jdalton
Copy link
Contributor

jdalton commented Feb 26, 2025

I'll merge and follow-up with a fix for the fails. No worries.

@jdalton jdalton merged commit b56ae70 into main Feb 26, 2025
9 of 15 checks passed
@jdalton jdalton deleted the dry_header branch February 26, 2025 17:42
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