The Approve choice should not be the first choice when running a deploy action.
In Terraform, when running an Apply, you must type yes to apply changes. This means the change will be canceled if you press enter accidentally. Here, with CDKTF, it will be applied, which can be very dangerous.
If you agree, I think we need to change the choice order here.
I'll be happy to make a PR to improve it.
References
No response
Help Wanted
[X] I'm interested in contributing a fix myself
Community Note
Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or other comments that do not add relevant new information or questions, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment
Description
The
Approvechoice should not be the first choice when running adeployaction. In Terraform, when running an Apply, you must typeyesto apply changes. This means the change will be canceled if you press enter accidentally. Here, with CDKTF, it will be applied, which can be very dangerous.If you agree, I think we need to change the choice order here. I'll be happy to make a PR to improve it.
References
No response
Help Wanted
Community Note