Skip to content

Ask user to confirm before development data deletion#4

Open
sayravai wants to merge 1 commit intoapluslms:masterfrom
sayravai:confirm-data-deletion
Open

Ask user to confirm before development data deletion#4
sayravai wants to merge 1 commit intoapluslms:masterfrom
sayravai:confirm-data-deletion

Conversation

@sayravai
Copy link
Copy Markdown

@sayravai sayravai commented Mar 2, 2026

Description

I have accidentally deleted my generated submissions too many times, so I was wondering if we could add a confirmation before doing that...

@ihalaij1 ihalaij1 force-pushed the confirm-data-deletion branch from cb76ee9 to 667edcd Compare March 31, 2026 09:04
@ihalaij1 ihalaij1 self-requested a review March 31, 2026 09:07
@ihalaij1
Copy link
Copy Markdown
Contributor

ihalaij1 commented Mar 31, 2026

@sayravai I made some more changes, so that you don't need to wait until the containers have been stopped to confirm with y/N. CTRL + C or Q are now the only ways to quit the script, and output is no longer printed after the script has exited (no more garbled output where the terminal looks messed up after quitting the script).

Could you test these changes and see what you think?

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