Skip to content

ci: add windows build - #332

Draft
gballet wants to merge 1 commit into
mainfrom
ci-add-windows-build
Draft

ci: add windows build#332
gballet wants to merge 1 commit into
mainfrom
ci-add-windows-build

Conversation

@gballet

@gballet gballet commented Oct 26, 2025

Copy link
Copy Markdown
Contributor

No description provided.

Copilot AI review requested due to automatic review settings October 26, 2025 20:41

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds Windows to the CI build matrix to ensure the project is tested across all major operating systems (Linux, macOS, and Windows).

Key Changes:

  • Extended the CI build matrix to include windows-latest alongside existing Ubuntu and macOS builds

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@gballet

gballet commented Oct 26, 2025

Copy link
Copy Markdown
Contributor Author

This is not going to happen right off the bat:

C:\Users\runneradmin\AppData\Local\zig\p\rocksdb-9.7.4-z_CUTmO5AAD0CQ2ZvShSDZHjC2x9MKrTnpvbNAIU7ah0\build.zig:473:9: 0x7ff75aec390b in buildRocksDB (build.exe.obj)
        @panic("TODO: support windows!");

@gballet
gballet marked this pull request as draft October 27, 2025 18:33
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