Skip to content

fix: deletion confirmation order in bss, cloud-init, and smd#99

Open
synackd wants to merge 3 commits into
mainfrom
fix/deletion-confirmation-order
Open

fix: deletion confirmation order in bss, cloud-init, and smd#99
synackd wants to merge 3 commits into
mainfrom
fix/deletion-confirmation-order

Conversation

@synackd

@synackd synackd commented Jun 9, 2026

Copy link
Copy Markdown
Collaborator

Checklist

  • My code follows the style guidelines of this project
  • I have added/updated comments where needed
  • I have added tests that prove my fix is effective or my feature works
  • I have run make test (or equivalent) locally and all tests pass
  • DCO Sign-off: All commits are signed off (git commit -s) with my real name and email
  • REUSE Compliance:
    • Each new/modified source file has SPDX copyright and license headers
    • Any non-commentable files include a <filename>.license sidecar
    • All referenced licenses are present in the LICENSES/ directory

Description

Move --no-confirm check for delete subcommands for bss, cloud-init, and smd commands to the top of the command logic to prompt the user before client actions are taken. This means the user will be prompted to confirm deletion before the token is handled.

Type of Change

  • Bug fix
  • New feature
  • Breaking change
  • Documentation update

For more info, see Contributing Guidelines.

@synackd synackd force-pushed the fix/deletion-confirmation-order branch from 0533169 to 67b6c27 Compare June 15, 2026 07:16
synackd added 3 commits June 15, 2026 02:59
Signed-off-by: Devon Bautista <17506592+synackd@users.noreply.github.com>
Signed-off-by: Devon Bautista <17506592+synackd@users.noreply.github.com>
Signed-off-by: Devon Bautista <17506592+synackd@users.noreply.github.com>
@synackd synackd force-pushed the fix/deletion-confirmation-order branch from 67b6c27 to 8184b01 Compare June 15, 2026 08:59
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.

1 participant