delete validation added #13
Conversation
There was a problem hiding this comment.
Pull request overview
This pull request adds an interactive confirmation prompt for deletion operations in the Sonar Docker image inspector tool. The change improves safety by requiring explicit user confirmation before executing potentially destructive operations.
- Adds interactive confirmation prompt for deletion operations (
--untagged-delete,--delete-all,--delete-tag) - Introduces
--forceflag to bypass the confirmation prompt for automated workflows - Removes duplicate argument parser code block that existed in the main function
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| console.print("[red] - Deleting: ALL detected manifest lists[/red]") | ||
| if args.delete_tag: | ||
| console.print(f"[red] - Deleting: Manifest lists with tag '{args.delete_tag}'[/red]") | ||
|
|
There was a problem hiding this comment.
Trailing whitespace detected at the end of this line. Remove the extra spaces for cleaner code formatting.
| args = parser.parse_args() | ||
| logger.info(f"Arguments: {args}") | ||
|
|
||
| # Interactive Confirmation for Deletion |
There was a problem hiding this comment.
Consider adding a comment above this block to explain the purpose of the interactive confirmation. For example: "Prompt user for confirmation before executing any deletion operations unless --force flag is provided."
| # Interactive Confirmation for Deletion | |
| # Prompt user for confirmation before executing any deletion operations unless --force flag is provided |
📄 Summary
Briefly describe the purpose of this pull request. What problem does it solve or feature does it add?
🔍 Related Issues
Link to any related GitHub issues (e.g.,
Fixes #12,Closes #34):🧪 Type of Change
Please check the relevant type tag for this PR title:
[FIX]Bug fix[NEW]New thing[REFACTOR]Internal changes such as code restructuring or optimization that does not alter functionality[DOC]Documentation-only changes[CHORE]Maintenance, cleanup, or CI configuration🧪 How Has This Been Tested?
Describe how you tested your changes. Include CI runs, local tests, manual verification, or screenshots if applicable.
📸 Screenshots (if applicable)
If UI or logs are affected, include before/after screenshots or output.
✅ Checklist