Skip to content

Commit

Permalink
chore: add second confirmation for container data loss (#1113)
Browse files Browse the repository at this point in the history
We have observed some users not realizing container data is lost when doing delete.
This change does not modify scripted behaviour, as -f / force toggles maintains same logic.
  • Loading branch information
jramosf authored Aug 19, 2024
1 parent 4816fd1 commit 426ccda
Showing 1 changed file with 4 additions and 0 deletions.
4 changes: 4 additions & 0 deletions cmd/delete.go
Original file line number Diff line number Diff line change
Expand Up @@ -28,6 +28,10 @@ If you simply want to reset the Kubernetes cluster, run 'colima kubernetes reset
if !y {
return nil
}
yy := cli.Prompt("\033[31m\033[1mthis will delete ALL container data. Are you sure you want to continue")
if !yy {
return nil
}
}

return newApp().Delete()
Expand Down

0 comments on commit 426ccda

Please sign in to comment.