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.

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

Delete existing release asset before releasing #29

Merged
merged 1 commit into from Aug 1, 2021

Conversation

ekangmonyet
Copy link
Contributor

We're using only one tag for release (latest), so we have to either replace it correctly or delete it before uploading, otherwise it will fail (https://github.com/ekangmonyet/lkmpg/runs/3213455232):

Error: Validation Failed: {"resource":"ReleaseAsset","code":"already_exists","field":"name"}

One possible solution is to use mknejp/delete-release-assets to delete existing asset. If action-gh-release supports upserting (softprops/action-gh-release#134), this should only be a temporary remedy. Warning from the delete-release-assets repository:

WARNING
Because this action has the power to delete any asset from any release in the repository (via the access token) be extra careful that you specify the correct tag!

Since it is not really urgent, we may wait for the patch for action-gh-release as well.

Use mknejp/delete-release-assets to delete existing asset.
This approach might not be optimal if softprops/action-gh-release
supports upserting [1].

[1] softprops/action-gh-release#134
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.

None yet

2 participants