Skip to content

Commit

Permalink
Remove short flag from history purge command. (#5242)
Browse files Browse the repository at this point in the history
The command `minder history purge` supports specifying a custom batch
size via `--batch-size/-s` option. The short version `-s` overlaps
with the short version of `--db-sslmode`, which is a root-level option.

This change solves the overlap by removing the short option from
`minder history purge`.
  • Loading branch information
blkt authored Jan 2, 2025
1 parent d3cc9b6 commit dee36be
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion cmd/server/app/history_purge.go
Original file line number Diff line number Diff line change
Expand Up @@ -169,6 +169,6 @@ func deleteEvaluationHistory(

func init() {
historyCmd.AddCommand(historyPurgeCmd)
historyPurgeCmd.Flags().UintP("batch-size", "s", 1000, "Size of the deletion batch")
historyPurgeCmd.Flags().UintP("batch-size", "", 1000, "Size of the deletion batch")
historyPurgeCmd.Flags().Bool("dry-run", false, "Avoids deleting, printing out details about the operation")
}

0 comments on commit dee36be

Please sign in to comment.