Skip to content

Fix manifest.json artifact upload in workflow - #1771

Merged
EnriqueL8 merged 3 commits into
hyperledger-firefly:mainfrom
dan-r:fix-release-asset-permissions
Sep 22, 2026
Merged

EnriqueL8 merged 3 commits 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

@EnriqueL8
EnriqueL8 merged commit 266549c into hyperledger-firefly:main Sep 22, 2026
6 checks passed
@dan-r
dan-r deleted the fix-release-asset-permissions branch September 22, 2026 14:26
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