Skip to content

Replace softprops/action-gh-release with direct GH api create release…#304

Merged
dbernstein merged 2 commits intomainfrom
replace-softprops-action-with-direct-release-gh-api-calls
Apr 14, 2026
Merged

Replace softprops/action-gh-release with direct GH api create release…#304
dbernstein merged 2 commits intomainfrom
replace-softprops-action-with-direct-release-gh-api-calls

Conversation

@dbernstein
Copy link
Copy Markdown
Contributor

… call

Why are we doing this? (w/ JIRA link if applicable)
We're trying to get the jira-release-sync workflow to trigger on a "publish" event.

How should this be tested? / Do these changes have associated tests?
Should be tested during next release.
Dependencies for merging? Releasing to production?
No.

Did someone actually run this code to verify it works?
Not yet: but it was taken from ios-core where it has run successfully.

@dbernstein dbernstein requested a review from a team April 14, 2026 18:16
Copy link
Copy Markdown
Contributor

@io7m io7m left a comment

Choose a reason for hiding this comment

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

GitHub seems unhappy with this. Comes up with an error in the YAML viewer.

Comment thread .github/workflows/android-release.yml Outdated
uses: softprops/action-gh-release@v2
- name: Create or update release
env:
GITHUB_TOKEN: ${{ GITHUB_TOKEN }}
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.

I think that should be secrets.CI_GITHUB_ACCESS_TOKEN.

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.

yeah - I wasn't sure since ios-core wasn't using it. But it's probably better to go with the ci token.

Copy link
Copy Markdown
Contributor

@io7m io7m left a comment

Choose a reason for hiding this comment

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

It seems happy with that. 🚀

@dbernstein dbernstein merged commit 6f15417 into main Apr 14, 2026
1 check failed
@dbernstein dbernstein deleted the replace-softprops-action-with-direct-release-gh-api-calls branch April 14, 2026 18:46
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