Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

CLI: --exclusively is -e for synth and deploy, but -x for destroy #2367

Closed
rix0rrr opened this issue Apr 24, 2019 · 2 comments · Fixed by #3216 or MechanicalRock/tech-radar#14 · May be fixed by MechanicalRock/cdk-constructs#5, MechanicalRock/cdk-constructs#6 or MechanicalRock/cdk-constructs#7
Closed

CLI: --exclusively is -e for synth and deploy, but -x for destroy #2367

rix0rrr opened this issue Apr 24, 2019 · 2 comments · Fixed by #3216 or MechanicalRock/tech-radar#14 · May be fixed by MechanicalRock/cdk-constructs#5, MechanicalRock/cdk-constructs#6 or MechanicalRock/cdk-constructs#7
Labels
bug This issue is a bug. package/tools Related to AWS CDK Tools or CLI

Comments

@rix0rrr
Copy link
Contributor

rix0rrr commented Apr 24, 2019

Describe the bug

cdk destroy -e some-stack

should EXCLUSIVELY destroy some-stack, but in fact it wants to destroy ALL STACKS in the application. Oops.
@rix0rrr rix0rrr added bug This issue is a bug. package/tools Related to AWS CDK Tools or CLI labels Apr 24, 2019
@BDQ
Copy link
Contributor

BDQ commented May 14, 2019

Did this not change to -x ?

@rix0rrr
Copy link
Contributor Author

rix0rrr commented Jun 26, 2019

You are frigging right.

yargs should have told me about that, I feel :x

@rix0rrr rix0rrr changed the title Toolkit: 'cdk destroy -e stack' wants to destroy everything CLI: --exclusively is -e for synth and deploy, but -x for destroy Jun 26, 2019
rix0rrr added a commit that referenced this issue Jul 5, 2019
@rix0rrr
fix(cli): exclusively is also '-e' for destroy
c32e477 
Make the short option for `--exclusively` always `-e` (used to be `-e`
for `synth` and `deploy`, but `-x` for `destroy).

Fixes #2367.
@rix0rrr rix0rrr mentioned this issue Jul 5, 2019
Merged
eladb pushed a commit that referenced this issue Jul 7, 2019
@rix0rrr
fix(cli): exclusively is also '-e' for destroy (#3216)
d6f5207 
Make the short option for `--exclusively` always `-e` (used to be `-e`
for `synth` and `deploy`, but `-x` for `destroy).

Fixes #2367.
This was referenced Aug 22, 2019
Open
Open
Open
Open
Open
This was referenced Dec 12, 2019
Closed
Closed
Closed
Merged
Closed
Open
Open
Open
Open
Open
Open
Open
Open
This was referenced Jan 20, 2020
Closed
Closed
@NOUIY NOUIY mentioned this issue Sep 24, 2024
Open
This was referenced Sep 24, 2024
Open
Open
Open
Open
Open
Open
Open
Open
Open
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug This issue is a bug. package/tools Related to AWS CDK Tools or CLI
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix(cli): exclusively is also '-e' for destroy aws/aws-cdk
[Snyk] Upgrade @aws-cdk/aws-iam from 0.22.0 to 0.39.0 MechanicalRock/tech-radar
[Snyk] Upgrade @aws-cdk/aws-codebuild from 0.24.1 to 0.39.0 MechanicalRock/cdk-constructs
[Snyk] Upgrade @aws-cdk/aws-codepipeline from 0.24.1 to 0.39.0 MechanicalRock/cdk-constructs
[Snyk] Upgrade @aws-cdk/aws-s3 from 0.24.1 to 0.39.0 MechanicalRock/cdk-constructs
[Snyk] Upgrade @aws-cdk/aws-cloudfront from 0.15.2 to 0.39.0 MechanicalRock/cdk-products
[Snyk] Upgrade @aws-cdk/aws-codebuild from 0.15.2 to 0.39.0 MechanicalRock/cdk-products
[Snyk] Upgrade @aws-cdk/aws-codecommit from 0.15.2 to 0.39.0 MechanicalRock/cdk-products
[Snyk] Upgrade @aws-cdk/aws-codepipeline from 0.15.2 to 0.39.0 MechanicalRock/cdk-products
[Snyk] Upgrade @aws-cdk/aws-cloudformation from 0.0.0 to 0.39.0 NOUIY/aws-solutions-constructs
2 participants
@BDQ @rix0rrr