beetbox / beetbox/beets

beet remove --delete should NOT default to "yes"

Open Beginner friendly
#5,845 9 comments 0 reactions 0 assignees View on GitHub
needinfo remove-command
Dominant language
Python
Stars
15.7k
Forks
2.1k
Avg merge
4d 21h
Merged PRs (30d)
31

Description

```bash
beet remove --delete albumartist:'miles' album:'porgy'
...
Really DELETE 15 files? (Yes/no/select)
```

I pressed enter here and it deleted the files! Seems like if it's going to warn about a destructive action, it should also default to NO, not YES.

Contributor guide

Open the contributing guide

Research direction

Start at the confirmation prompt used by `beet remove --delete` and reproduce the command shown in the issue. The change is done when pressing Enter keeps the files intact, while an explicit affirmative response still permits deletion; add or update coverage for this prompt behavior if the existing test location is found.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
cli
Issue type
Bug
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
62/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.