Skip to content

Conversation

@igitur
Copy link

@igitur igitur commented Jul 2, 2025

Add Windows 64b build to CI

@bart6114
Copy link
Owner

bart6114 commented Jul 6, 2025

Thanks for this @igitur.
Where you able to test the binaries? I don't have access to a win machine.

We should prolly run tests against the architecture as well 🤔

@igitur
Copy link
Author

igitur commented Jul 6, 2025

I've been manually building the Windows binaries all the time and been using it for semi-production work. I wrap the cheek command in nssm so that it launches automatically when my server reboots.

But sure, after you release, I'll check again.

Will look into the tests too and update PR.

@bart6114
Copy link
Owner

bart6114 commented Jul 7, 2025

WDYT, should we add something like this to the tests job? And include it in this pr?

  tests:
    runs-on: ${{ matrix.os }}
    strategy:
      matrix:
        os: [ubuntu-latest, windows-latest]

@igitur
Copy link
Author

igitur commented Jul 7, 2025

Ok, it's a bit more work than that, because a lot of the test data includes sh -c ... kind of commands.

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