-
-
Notifications
You must be signed in to change notification settings - Fork 479
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
Support release asset patching behavior to overwrite existing release assets #45
Comments
Would love this |
I have the same use-case, updating the nightly into the same Release, including replacing the assets. @ppaulweber how did you get around this? Did you use another action to delete the release as suggested in #47? |
@proddy yes, I use another action to erase the existing release and then create a new one, but I would rather like to "re-release" to the same tag as proposed in this issue. |
I just ran into this as well. |
I used delete-tag-and-release as in https://github.com/proddy/EMS-ESP/blob/dev/.github/workflows/build_firmware.yml sloppy, but it kinda works |
Are there any updates? |
@larshp take a look at https://github.com/ncipollo/release-action |
upserting asserts this is now a default |
Hello, I've just discovered that the overwrite of already existing release assets of a existing release tag/name is not handled properly. The GitHub action just fails silent with not handled promise rejection warnings (why not errors?) and returns a success state, which is incorrect and should be failing!
The problem occurs in one of my nightly build and release GitHub workflows, which updates the same release tag/name with rebuild binaries, but the binaries are not updated at all. The following log message depicts the not handled promise rejection warning:
The issue and feature request is similar to #42, which describes a merging of assets, but not explicit overwrite. Maybe an option (e.g.
asset
) can be introduced in the GitHub action configuration to set different GitHub release patch behaviors.asset: create
represents the current and default behavior, but shall fail if the asset already existsasset: patch
represents a new behavior to overwrite only existing assetsasset: merge
represents a new behavior to overwrite existing and add new ones as well, so a combination ofcreate
andpatch
, which shall cover the issue in Support release asset merging #42 as well.A workflow example configuration would look like:
The text was updated successfully, but these errors were encountered: