Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ci: fix a bug that prebuilt binaries aren't published to GitHub Releases #48

Merged
merged 1 commit into from
Nov 10, 2023
Merged

ci: fix a bug that prebuilt binaries aren't published to GitHub Releases #48

merged 1 commit into from
Nov 10, 2023

Conversation

suzuki-shunsuke
Copy link
Contributor

@suzuki-shunsuke suzuki-shunsuke commented Nov 3, 2023

omekasy v1.2.1 and v1.2.2 don't have prebuilt binaries.

https://github.com/ikanago/omekasy/releases/tag/v1.2.2

image

Prebuilt binaries were built in CI, but they weren't published because the input of softprops/action-gh-release files is empty.

https://github.com/ikanago/omekasy/actions/runs/6284001639/job/17064954741

Run softprops/action-gh-release@v1
  with:
    token: ***
  env:
    PROJECT_NAME: omekasy
    PKG_PATH: omekasy-v1.2.2-x86_64-unknown-linux-gnu.tar.gz
    GITHUB_TOKEN: ***
πŸ‘©β€πŸ­ Creating new GitHub release for tag v1.2.2...
πŸ€”  not include valid file.
πŸŽ‰ Release ready at https://github.com/ikanago/omekasy/releases/tag/v1.2.2

omekasy v1.2.1 and v1.2.2 don't have prebuilt binaries.

https://github.com/ikanago/omekasy/releases/tag/v1.2.2

Prebuilt binaries were built in CI, but they weren't published because the input of softprops/action-gh-release `files` is empty.

https://github.com/ikanago/omekasy/actions/runs/6284001639/job/17064954741

```
Run softprops/action-gh-release@v1
  with:
    token: ***
  env:
    PROJECT_NAME: omekasy
    PKG_PATH: omekasy-v1.2.2-x86_64-unknown-linux-gnu.tar.gz
    GITHUB_TOKEN: ***
πŸ‘©β€πŸ­ Creating new GitHub release for tag v1.2.2...
πŸ€”  not include valid file.
πŸŽ‰ Release ready at https://github.com/ikanago/omekasy/releases/tag/v1.2.2
```
@@ -66,7 +66,7 @@ jobs:
PKG_SUFFIX=".tar.gz" ; case "${{ matrix.job.target }}" in *-pc-windows-*) PKG_SUFFIX=".zip" ;; esac;
PKG_BASENAME="${PROJECT_NAME}-v${{ steps.project_version.outputs.PROJECT_VERSION }}-${{ matrix.job.target }}"
PKG_PATH="${PKG_BASENAME}${PKG_SUFFIX}"
echo "PKG_PATH=${PKG_PATH}" >> "${GITHUB_ENV}"
echo "PKG_PATH=${PKG_PATH}" >> "${GITHUB_OUTPUT}"
Copy link
Contributor Author

Choose a reason for hiding this comment

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

https://docs.github.com/en/actions/using-workflows/workflow-commands-for-github-actions#setting-an-output-parameter

uses: softprops/action-gh-release@v1
with:
files: ${{ steps.create_tarball.outputs.PKG_PATH }}

@suzuki-shunsuke suzuki-shunsuke marked this pull request as ready for review November 3, 2023 04:10
@suzuki-shunsuke
Copy link
Contributor Author

This bug was caused by the following changes.

@ikanago
Copy link
Owner

ikanago commented Nov 10, 2023

Thank you for your first contribution!
This is really helpful fix as I could not figure out why the prebuilt binaries are not shipped.
I'm going to release v1.2.3 soon.

@ikanago ikanago merged commit 3ada6d0 into ikanago:main Nov 10, 2023
2 checks passed
@ikanago
Copy link
Owner

ikanago commented Nov 10, 2023

v1.2.3 and the prebuilt binaries are successfully shipped!
I appreciate your contribution again πŸ™
https://github.com/ikanago/omekasy/releases/tag/v1.2.3

@suzuki-shunsuke suzuki-shunsuke deleted the ci-publish-prebuilt-binaries branch November 10, 2023 12:27
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.

None yet

2 participants