Skip to content
This repository was archived by the owner on Jan 2, 2025. It is now read-only.

Conversation

@rsdy
Copy link
Contributor

@rsdy rsdy commented Jun 30, 2023

Would be helpful to get an acceptance test on all platforms before merging.

Copy link
Contributor

@calyptobai calyptobai left a comment

Choose a reason for hiding this comment

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

I've left a few comments regarding some of the internals. I can test this PR out on Windows after any changes to make sure that it continues to work

Copy link
Contributor

@ggordonhall ggordonhall left a comment

Choose a reason for hiding this comment

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

The server does not start on the aarch64-apple-darwin build.

@rsdy rsdy force-pushed the windows-builds2 branch 6 times, most recently from ce65f31 to 48beee9 Compare July 4, 2023 08:26
@rsdy rsdy force-pushed the windows-builds2 branch from 03fcea0 to e2a7da3 Compare July 4, 2023 10:00
@ggordonhall ggordonhall dismissed their stale review July 4, 2023 15:15

It now works

@rsdy rsdy merged commit e0a6739 into main Jul 4, 2023
@rsdy rsdy deleted the windows-builds2 branch July 4, 2023 15:22
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants