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

[CI/CD] Update automatic release action to actively maintained one #1207

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

taysta
Copy link
Contributor

@taysta taysta commented Feb 13, 2024

https://github.com/marvinpinto/action-automatic-releases is currently used to download the artifacts and to create the "latest" release. It is in an abandoned state using Node 12 and will stop working entirely quite soon.

https://github.com/softprops/action-gh-release is an actively maintained action that achieves the same purpose. It is likely going to have a v2 soon, which will include a bump to Node 20, which has already been committed (as Node 16 depreciation warnings have now begun on actions).

Known problem with this runner:
It successfully uploads the artifacts and creates the latest release, if a latest release exists, it re-uploads the files to that pre-existing release, instead of re-creating the "latest" release as the current action does. I have only tested this on v1 so far, and will shortly run some tests to see if this behavior persists on the latest commit of the action.

Update:
https://github.com/crowbarmaster/GH-Automatic-Releases is a suitable replacement, it functions 1:1 to the previous implementation and has already been updated to Node 20.

@taysta taysta requested a review from a team as a code owner February 13, 2024 11:12
@taysta taysta marked this pull request as draft February 13, 2024 11:13
@taysta taysta marked this pull request as ready for review February 23, 2024 05:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants