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

upkeep/wp-deploy: Move from actions/upload-release-asset to softprops/action-gh-release #262

Merged
merged 3 commits into from
Aug 14, 2024

Conversation

Sidsector9
Copy link
Member

@Sidsector9 Sidsector9 commented Jul 19, 2024

Description of the Change

actions/upload-release-asset has been deprecated and softprops/action-gh-release is the recommended replacement.

How to test the Change

Verify the ZIP attached in a release.

Changelog Entry

Changed - Move from actions/upload-release-asset to softprops/action-gh-release Github action.

@Sidsector9 Sidsector9 requested a review from jeffpaul as a code owner July 19, 2024 15:19
@github-actions github-actions bot added this to the 1.2.0 milestone Jul 19, 2024
Copy link

@Sidsector9 thanks for the PR! Could you please fill out the PR template with description, changelog, and credits information so that we can properly review and merge this?

@github-actions github-actions bot added the needs:feedback This requires reporter feedback to better understand the request. label Jul 19, 2024
@Sidsector9 Sidsector9 assigned Sidsector9 and unassigned Sidsector9 Jul 19, 2024
@@ -21,11 +21,8 @@ jobs:
SVN_PASSWORD: ${{ secrets.SVN_PASSWORD }}
SVN_USERNAME: ${{ secrets.SVN_USERNAME }}
- name: Upload release asset
uses: actions/upload-release-asset@v1
uses: softprops/action-gh-release@v2
Copy link
Member

Choose a reason for hiding this comment

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

Copy link
Member Author

Choose a reason for hiding this comment

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

@jeffpaul we don't need to update the actions you mentioned as this change is only for the asset we attach to the release. The plugin build zip and asset deploy actions are unrelated.

Though we need to update the examples here:

https://github.com/10up/action-wordpress-plugin-deploy/blob/9273e6df8583dadbfd50001908c11c30009a7db4/examples/deploy-on-publishing-a-new-release-and-attach-a-zip-file-to-the-release.yml#L82-L100

Copy link
Member

Choose a reason for hiding this comment

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

@Sidsector9 mind pushing a fix for the action examples then?

Copy link
Member Author

Choose a reason for hiding this comment

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

@jeffpaul jeffpaul merged commit 844c7c6 into develop Aug 14, 2024
12 of 14 checks passed
@jeffpaul jeffpaul deleted the upkeep/wp-deploy branch August 14, 2024 19:28
@dkotter dkotter modified the milestones: 1.2.0, 1.1.3 Nov 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs:feedback This requires reporter feedback to better understand the request.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants