Skip to content

fix: replace --dry-run flag with --confirm for prune-archived #12

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
utahcon merged 1 commit into from
Mar 9, 2026
Merged

fix: replace --dry-run flag with --confirm for prune-archived #12

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
18 changes: 9 additions & 9 deletions cmd/prune_archived.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -11,9 +11,9 @@ import (
)

var (
pruneOrg string
prunePath string
dryRun bool
pruneOrg string
prunePath string
pruneConfirm bool
)

var pruneArchivedCmd = &cobra.Command{
Expand All @@ -23,8 +23,8 @@ var pruneArchivedCmd = &cobra.Command{
Any directory that corresponds to an archived repository in the specified organization
will be removed.

By default runs in dry-run mode (--dry-run) so you can preview what would be deleted.
Pass --dry-run=false to actually remove directories.`,
By default runs in dry-run mode so you can preview what would be deleted.
Pass --confirm to actually remove directories.`,
RunE: func(cmd *cobra.Command, args []string) error {
return runPruneArchived(cmd.Context())
},
Expand All @@ -34,7 +34,7 @@ Pass --dry-run=false to actually remove directories.`,
func init() {
pruneArchivedCmd.Flags().StringVarP(&pruneOrg, "org", "o", "", "GitHub organization to check repositories against (required)")
pruneArchivedCmd.Flags().StringVarP(&prunePath, "path", "p", "", "Local path containing cloned repositories (required)")
pruneArchivedCmd.Flags().BoolVar(&dryRun, "dry-run", true, "Preview what would be removed without deleting anything")
pruneArchivedCmd.Flags().BoolVar(&pruneConfirm, "confirm", false, "Actually remove directories (without this flag, runs in dry-run mode)")

_ = pruneArchivedCmd.MarkFlagRequired("org")
_ = pruneArchivedCmd.MarkFlagRequired("path")
Expand Down Expand Up @@ -87,7 +87,7 @@ func runPruneArchived(ctx context.Context) error {

dirPath := filepath.Join(absPath, name)

if dryRun {
if !pruneConfirm {
fmt.Printf("[dry-run] would remove: %s\n", dirPath)
removedCount++
continue
Expand All @@ -102,10 +102,10 @@ func runPruneArchived(ctx context.Context) error {
removedCount++
}

if dryRun {
if !pruneConfirm {
fmt.Printf("\nDry-run complete: %d archived repositories would be removed\n", removedCount)
if removedCount > 0 {
fmt.Println("Run with --dry-run=false to actually remove them")
fmt.Println("To actually remove them, re-run with the --confirm flag")
}
} else {
fmt.Printf("\nSummary: Removed %d archived repositories", removedCount)
Expand Down