Skip to content

CI: getting started with GitHub Actions#37

Open
pzhlkj6612 wants to merge 5 commits intoantonmedv:masterfrom
pzhlkj6612:ci
Open

CI: getting started with GitHub Actions#37
pzhlkj6612 wants to merge 5 commits intoantonmedv:masterfrom
pzhlkj6612:ci

Conversation

@pzhlkj6612
Copy link

Hello!


In README.md, we have:

...
Or download prebuilt binary from releases.

However, only the "v1.0.0" release contains pre-built binaries, so I've made a workflow to make snapshot releases.

Please note that not all GOOS/GOARCH pairs are supported by GitHub Actions, so it may not be possible to build for every platform supported by Go. The android/arm64 build works on my phone.

@antonmedv
Copy link
Owner

As I release not very often I decided not to go with CI.

But countdown got much more popular than anticipated. Probably a good time for a CI

Let’s implement something similar like in my other projects: https://github.com/antonmedv/llama/blob/master/.github/workflows/release.mjs

@pzhlkj6612
Copy link
Author

Oh, it seems that I've just made a few things about releasing a new countdown... I need to learn the workflow in antonmedv/llama first.

@pzhlkj6612 pzhlkj6612 marked this pull request as draft April 17, 2023 18:03
- No Android, too difficult.
- "install.sh" does not make so much sense on Windows.
@pzhlkj6612
Copy link
Author

Hey!

Let’s implement something similar like in my other projects: https://github.com/antonmedv/llama/blob/master/.github/workflows/release.mjs

Done. I've been using https://github.com/antonmedv/walk/blob/c78d2116a868f5471171c0f5a8d36243e44298eb/.github/scripts/build.mjs as the base.

For now:

.

@pzhlkj6612 pzhlkj6612 marked this pull request as ready for review June 17, 2025 17:18
run: |
set -euo pipefail

npm install zx
Copy link
Owner

Choose a reason for hiding this comment

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

Npm install not needed for npx

Copy link
Author

Choose a reason for hiding this comment

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

You're right! 5f78309.

Co-authored-by: Anton Medvedev <anton@medv.io>
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