-
Notifications
You must be signed in to change notification settings - Fork 72
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.
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
CLI: implement delete cmd #330
Conversation
ecb95bc
to
9acc85e
Compare
Signed-off-by: Augustin Husson <husson.augustin@gmail.com>
Signed-off-by: Augustin Husson <husson.augustin@gmail.com>
9a257d1
to
62e196e
Compare
Signed-off-by: Augustin Husson <husson.augustin@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, just spotted few typos in some tests titles
ExpectedMessage: `resource "whatever" not managed`, | ||
}, | ||
{ | ||
Title: "not connected to anyAPI", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
any API
ExpectedMessage: "you are not connected to any API", | ||
}, | ||
{ | ||
Title: "delete all project", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
projects
`, | ||
}, | ||
{ | ||
Title: "delete multiples different resources from a file", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
multiple resources
Signed-off-by: Augustin Husson <husson.augustin@gmail.com>
cool, thanks @AntoineThebaud ! |
This PR is proposing a way to delete resources with the CLI. I'm printing here the help from the command, hoping it will help to understand how it is supposed to work and to describe this PR :).
This PR should be merged after #326 because it contains the code to manage file and stdin required for this command.