Skip to content

chore(publish): publish releases - #8648

Merged
notjaywu merged 1 commit into
developfrom
chore/publish-releases
Apr 28, 2025
Merged

chore(publish): publish releases#8648
notjaywu merged 1 commit into
developfrom
chore/publish-releases

Conversation

@notjaywu

Copy link
Copy Markdown
Contributor

No description provided.

@notjaywu
notjaywu force-pushed the chore/publish-releases branch from 7401ded to bc4e68b Compare April 25, 2025 08:11
@notjaywu notjaywu changed the title [WIP] chore(publish): publish releases chore(publish): publish releases Apr 25, 2025
@notjaywu
notjaywu marked this pull request as ready for review April 25, 2025 08:13
@notjaywu
notjaywu requested review from a team, balugeorge and Copilot April 25, 2025 09:41

Copilot AI left a comment

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.

Pull Request Overview

This PR refactors the release publishing workflow by eliminating the legacy artifact downloader and artifact uploader actions and instead leverages the GitHub CLI for release verification, asset download, and release editing.

  • Removed the robinraju/release-downloader action and its associated steps.
  • Added a check to verify if the release exists and download its assets using the gh CLI.
  • Replaced the tag/release and artifact upload steps with a unified "Publish release" step via gh release edit.
Comments suppressed due to low confidence (2)

.github/workflows/release-publish.yml:141

  • [nitpick] Consider aligning the token environment variable name with previous steps (e.g. using 'GITHUB_TOKEN') for consistency and clarity across the workflow.
GH_TOKEN: ${{ secrets.GITHUB_TOKEN }}

.github/workflows/release-publish.yml:133

  • The new 'Publish release' step no longer handles artifact uploads, which were previously performed by the release upload step. Please verify that this change is intentional and that artifact uploading is not required.
- name: Publish release

@notjaywu
notjaywu force-pushed the chore/publish-releases branch from bc4e68b to 32150db Compare April 28, 2025 06:45
@notjaywu
notjaywu merged commit 2cd0317 into develop Apr 28, 2025
@notjaywu
notjaywu deleted the chore/publish-releases branch April 28, 2025 06:58
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.

3 participants