Skip to content

Fix release action#158

Open
prettycoder wants to merge 6 commits intomain_nhfrom
fix_release_action
Open

Fix release action#158
prettycoder wants to merge 6 commits intomain_nhfrom
fix_release_action

Conversation

@prettycoder
Copy link
Copy Markdown
Contributor

No description provided.

@prettycoder prettycoder requested a review from cheempz April 29, 2022 19:15
Copy link
Copy Markdown
Contributor

@cheempz cheempz left a comment

Choose a reason for hiding this comment

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

Thanks @prettycoder, changes LGTM but I do have a question about the "create GH Release" step.

-X POST \
-H "Accept: application/vnd.github.v3+json" \
https://api.github.com/repos/OWNER/REPO/releases \
-d '{"tag_name":"${{ steps.build.outputs.gem_version }}","name":"${{ steps.build.outputs.gem_version }}","body":"SHA256 checksum: ${{ steps.build.outputs.checksum }}","draft":true}'
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.

AFAICT actions/github-script is still maintained by GHA, so wondering why this change? Also, how are credentials being given by curl when making the call to GH API, and are OWNER/REPO getting set correctly?

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.

the create_release script is not maintained anymore and not working anymore
When I wrote the release notes I realized that this version is not working either, but it doesn't fail the action
Let me see if I can fix it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants