Add Windows 64b build to CI

[igitur]

Add Windows 64b build to CI

[bart6114]

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]

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]

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]

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

[igitur]

The full Windows CI is proving to be a bit challenging (see previous comments). But are you OK with at least producing the Windows binary without running the tests on Windows? The assumption then is that if the tests pass on Linux/Darwin, then the test would also have passed on Windows. At least this isn't a regresssion, because currently there is no Windows CI at all.

[igitur]

Ignore the previous comment. I now know that the Windows runners have Bash installed via Git for Bash. This PR should now run successfully and build cheek for Windows and run the test suite. I had to skip 2 tests for Windows though. I can look into refactoring those tests if required.

[bart6114]

Thanks a lot, please verify the new build artifacts when available as I don't have access to a win machine. When everything works, could you perhaps also add the build artifacts to the docs? Thanks!

[igitur]

Thanks, I downloaded the amd build from the new release. Everything works 100% except, cheek.exe version yields development instead of the actual version number. I will investigate later.