Skip to content

Add release workflow - #5

Open
sermuns wants to merge 2 commits into
dead10ck:masterfrom
sermuns:master
Open

Add release workflow#5
sermuns wants to merge 2 commits into
dead10ck:masterfrom
sermuns:master

Conversation

@sermuns

@sermunssermuns commented Sep 15, 2025

Copy link
Copy Markdown

This adds a GitHub action workflow that compiles the program for MacOS, Windows and various architectures of Linux.

I did a little tinkering with the git history to make the commits make the most sense, by adding the workflow before version 1.0.1. Let me know if this is stupid.

I also replaced = with # in CHANGELOG.md, let me know if there was a reason for this otherwise.

@dead10ckdead10ck left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Thanks for doing this; this mostly looks good to me. I'll test it out and merge when I get some time

Comment on lines +60 to +66
- name: Build | Build
if: ${{ !endsWith(matrix.target, '-unknown-linux-musl') }}
run: cargo build --release --target ${{ matrix.target }}

- name: Build | Build (musl)
if: endsWith(matrix.target, '-unknown-linux-musl')
run: cross build --release --target ${{ matrix.target }}

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

It looks like the only difference between these is the name? Not a big deal, just curious if I'm missing something.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

In musl it is using cross build !

Comment thread.github/workflows/release.yml
@sermuns

Copy link
Copy Markdown
Author

Any changes that need to be made?

@sermuns
sermunsforce-pushed the master branch 2 times, most recently from dcbe766 to 0a6f83bCompareJune 14, 2026 14:26
@sermuns

Copy link
Copy Markdown
Author

Sorry, was messing around modernizing the project, but that doesn't belong in this PR. I reset the changes to my original proposal!

Sign up for freeto 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

@sermuns@dead10ck