Skip to content

Fix manifest.json artifact upload in workflow - #1771

Open
dan-r wants to merge 1 commit into
hyperledger-firefly:mainfrom
dan-r:fix-release-asset-permissions
Open

Fix manifest.json artifact upload in workflow#1771
dan-r wants to merge 1 commit into
hyperledger-firefly:mainfrom
dan-r:fix-release-asset-permissions

Conversation

@dan-r

@dan-r dan-r commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

Changes workflow permissions to allow writing artifacts.

Also fixes leading comma issue and replaces the deprecated workflow.

Signed-off-by: Daniel Raper <daniel.raper@kaleido.io>
@dan-r
dan-r requested a review from a team as a code owner August 4, 2026 14:11
uses: actions/upload-release-asset@v1
env:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
uses: softprops/action-gh-release@v3

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

what is softprops? Why are we not using the official actions?

@dan-r dan-r Aug 5, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Its recommended by the GitHub team as a replacement for the discontinued action https://github.com/actions/upload-release-asset

Please note: This repository is currently unmaintained by a team of developers at GitHub. The repository is here and you can use it as an example, or in Actions. However please be aware that we are not going to be updating issues or pull requests on this repository.

Maintained Actions:

softprops/action-gh-release

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.

2 participants