Skip to content

Conversation

prettycoder
Copy link
Contributor

No description provided.

@prettycoder prettycoder requested a review from cheempz April 29, 2022 19:15
Copy link
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
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
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