Skip to content

Conversation

@AlecRust
Copy link
Contributor

@AlecRust AlecRust commented Jun 9, 2021

Description of the Change

The current "Deploy on publishing a new release and attach a ZIP file to the release" example uses actions/upload-release-asset which is deprecated.

Attaching files to an existing release doesn't seem that common in my research, the maintained GitHub release actions seem to focus on creating a release, optionally with files e.g. softprops/action-gh-release.

This adds a "Deploy on pushing a new tag and create release with attached ZIP" example to the README which is perhaps a more suitable example.

The implication with creating the release in an action is the changelog will be generated in the workflow too. I've omitted that from my example but would be happy to include if you like, perhaps there could be an /examples folder in this repo.

The current "Deploy on publishing a new release and attach a ZIP file to the release" example uses `actions/upload-release-asset` which is deprecated.

Attaching files to a previous release doesn't seem that common in my research, the maintained GitHub release actions seem to focus on creating a release, optionally with files e.g. `softprops/action-gh-release`.

This adds a "Deploy on pushing a new tag and create release with attached ZIP" example to the README which seems to be the latest approach.

The implication with creating the release in an action is the changelog will be generated in the action too. I've omitted that from my example but would be happy to include if you like, perhaps there could be an `/examples` folder in this repo.
@jeffpaul jeffpaul requested a review from helen June 9, 2021 22:02
@helen
Copy link
Collaborator

helen commented Aug 16, 2021

Hi there - I'm not sure I agree with your assessment that attaching files to releases after creation is uncommon, but perfectly happy to have more examples. I do think that at this point there probably should be an examples directory with different example workflows in there, so if you'd like to rework this to start that that would be great.

@jeffpaul jeffpaul added the needs:documentation This requires documentation. label Aug 20, 2021
@jeffpaul jeffpaul added this to the 2.1.0 milestone Aug 20, 2021
@jeffpaul jeffpaul modified the milestones: 2.1.0, 2.2.0 May 11, 2022
@jeffpaul
Copy link
Member

jeffpaul commented Jan 9, 2023

@AlecRust any chance you want to update the PR here to add your example as a workflow file in an examples directory with a linked reference from the readme perhaps?

@jeffpaul jeffpaul modified the milestones: 2.2.0, Future Release Jan 9, 2023
@jeffpaul jeffpaul added the help wanted Extra attention is needed label Apr 10, 2023
@jeffpaul jeffpaul added the needs:feedback This requires reporter feedback to better understand the request. label Apr 24, 2023
@jeffpaul jeffpaul mentioned this pull request Jan 3, 2024
4 tasks
@jeffpaul
Copy link
Member

jeffpaul commented Jan 3, 2024

@AlecRust note that I pulled your work from here into #143

@jeffpaul jeffpaul closed this Jan 3, 2024
@jeffpaul jeffpaul removed this from the Future Release milestone Jan 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

help wanted Extra attention is needed needs:documentation This requires documentation. needs:feedback This requires reporter feedback to better understand the request.

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants