Skip to content

crt: add confirm option#22

Open
UweSchwaeke wants to merge 2 commits intoclyso:mainfrom
UweSchwaeke:wip-crt-add-confirm-option
Open

crt: add confirm option#22
UweSchwaeke wants to merge 2 commits intoclyso:mainfrom
UweSchwaeke:wip-crt-add-confirm-option

Conversation

@UweSchwaeke
Copy link
Collaborator

@UweSchwaeke UweSchwaeke commented Jan 28, 2026

crt: introduce confirmation options

  • what:
    introduce --yes and --no options to crt subcommands that might ask for confirmation during execution.

  • why:
    allow the tool to run non-interactively. in scenarios like testing and automation, it is impossible to provide user input manually.

@UweSchwaeke UweSchwaeke requested a review from jecluis January 29, 2026 05:33
@UweSchwaeke UweSchwaeke force-pushed the wip-crt-add-confirm-option branch from d6b244f to e674794 Compare January 29, 2026 14:37
@UweSchwaeke UweSchwaeke force-pushed the wip-crt-add-confirm-option branch from e674794 to f6a33f7 Compare February 9, 2026 10:57
@UweSchwaeke UweSchwaeke force-pushed the wip-crt-add-confirm-option branch from f6a33f7 to f48371d Compare February 23, 2026 19:36
@UweSchwaeke UweSchwaeke force-pushed the wip-crt-add-confirm-option branch 4 times, most recently from 38670ec to d56fb3d Compare March 11, 2026 14:10
Copy link
Contributor

@jecluis jecluis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

needs to be rebased on top of main, the 'logs gc' patch needs to be dropped.

add option --yes to crt manifest release-notes is false by default.
if crt manifest release-notes is called with --yes, don't ask for confirmation to overwrite the release notes.

Signed-off-by: Uwe Schwaeke <uwe.schwaeke@clyso.com>
if --no is passed to crt patchset create, assume no description is added if the option -d is not passed.
if --yes is passed to crt patchset add, automatically add patches to the patchset instead of asking.
if --yes is passed to crt patchset publish, do not ask for confirmation to publish the patchset.
if --yes is passed to crt patchset remove, do not ask for confirmation to remove the patchset.

Signed-off-by: Uwe Schwaeke <uwe.schwaeke@clyso.com>
@UweSchwaeke UweSchwaeke force-pushed the wip-crt-add-confirm-option branch from d56fb3d to 88b4408 Compare March 12, 2026 11:34
@UweSchwaeke
Copy link
Collaborator Author

UweSchwaeke commented Mar 12, 2026

i rebased main.

@UweSchwaeke UweSchwaeke requested a review from jecluis March 12, 2026 11:36
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