Skip to content

#4258 Add sub command to delete asset from release#4416

Merged
samcoe merged 2 commits intocli:trunkfrom
m4ver1k:feat/4258
Jan 21, 2022
Merged

#4258 Add sub command to delete asset from release#4416
samcoe merged 2 commits intocli:trunkfrom
m4ver1k:feat/4258

Conversation

@m4ver1k
Copy link
Contributor

@m4ver1k m4ver1k commented Oct 2, 2021

Fixes #4258 add a sub command to delete assets

@Dstiers1191
Copy link

Dstiers1191 commented Oct 3, 2021 via email

@@ -0,0 +1,127 @@
package deleteasset
Copy link
Contributor Author

Choose a reason for hiding this comment

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

i'm not so happy with this package name 😓

@m4ver1k
Copy link
Contributor Author

m4ver1k commented Oct 5, 2021

@mislav can you please take a look at this PR ?

@mislav mislav self-assigned this Oct 6, 2021
@mislav mislav added the external pull request originating outside of the CLI core team label Dec 20, 2021
@zhangguanzhang
Copy link

any update?

@zhangguanzhang
Copy link

@mislav

@samcoe samcoe assigned samcoe and unassigned mislav Jan 14, 2022
@samcoe samcoe self-requested a review January 14, 2022 10:59
@m4ver1k m4ver1k requested a review from a team as a code owner January 20, 2022 10:55
Copy link
Contributor

@samcoe samcoe left a comment

Choose a reason for hiding this comment

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

@m4ver1k This looks great! I added a commit to just polish up a couple of small thing.

cc @mislav Since you were involved in the design of the command in the issue.

Copy link
Contributor

@mislav mislav left a comment

Choose a reason for hiding this comment

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

Looks great!

@samcoe samcoe merged commit bb3070a into cli:trunk Jan 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

external pull request originating outside of the CLI core team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Provide way to fully clean up a release via CLI

5 participants