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

Automatically build and release the CLI for the windows-386 build target #797

Merged
merged 3 commits into from
Feb 3, 2023

Conversation

timrogers
Copy link
Contributor

@timrogers timrogers commented Feb 2, 2023

This updates our release process to build each of our CLIs for x86 Windows devices (i.e. just about anything that isn't x64 or ARM!).

At this time, we don't have a way to test that these binaries work as part of our CI/CD process. Nevertheless, we think it's better to distribute a binary that should work than to offer nothing to users who want x86 support! If people run into any problems, they can create an issue and we will, of course, look into it.

Fixes #796.

  • Did you write/update appropriate tests
  • Release notes updated (if appropriate)
  • Appropriate logging output
  • Issue linked
  • Docs updated (or issue created)

…arget

This updates our release process to build each of our CLIs for x86 Windows devices (i.e. just about anything that isn't x64 or ARM!).

At this time, we don't have a way to test that these binaries work as part of our CI/CD process. Nevertheless, we think it's better to distribute a binary that should work than to offer nothing to users who want x86 support! If people run into any problems, they can create an issue and we will, of course, look into it.

Fixes #796.
@github-actions
Copy link

github-actions bot commented Feb 2, 2023

Integration Test Results

5 tests  ±0   5 ✔️ ±0   26m 21s ⏱️ + 10m 21s
1 suites ±0   0 💤 ±0 
1 files   ±0   0 ±0 

Results for commit 1450a0c. ± Comparison against base commit a4d4c84.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

github-actions bot commented Feb 2, 2023

Unit Test Results

640 tests   640 ✔️  20s ⏱️
    1 suites      0 💤
    1 files        0

Results for commit 1450a0c.

♻️ This comment has been updated with latest results.

@timrogers
Copy link
Contributor Author

@dylan-smith I'm seeing the Windows integration test fail - I guess I've broken something! - but looking at the test output, I can't find the line that is actually failing. I can just see that the exit code is wrong.

Could you take a look for me? Pretty sure I'm just being stupid... 😓

@github-actions
Copy link

github-actions bot commented Feb 3, 2023

Code Coverage

Package Line Rate Branch Rate Complexity Health
Octoshift 87% 72% 1014
bbs2gh 76% 75% 418
ado2gh 84% 78% 674
gei 84% 81% 569
Summary 84% (5870 / 6986) 76% (1399 / 1846) 2675

@timrogers
Copy link
Contributor Author

Ignore me! It passed with another try.

@timrogers timrogers merged commit 2a32819 into main Feb 3, 2023
@timrogers timrogers deleted the timrogers/windows-386 branch February 3, 2023 22:41
@chtzvt chtzvt mentioned this pull request Apr 25, 2024
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.

Compile our CLIs for non 64-bit non-ARM Windows machines (i.e. x86)
2 participants