Skip to content

fix(release): drop Windows quant-server.exe + add workflow_dispatch#45

Merged
unamedkr merged 1 commit intomainfrom
fix/release-windows-no-server
Apr 11, 2026
Merged

fix(release): drop Windows quant-server.exe + add workflow_dispatch#45
unamedkr merged 1 commit intomainfrom
fix/release-windows-no-server

Conversation

@unamedkr
Copy link
Copy Markdown
Collaborator

The Windows build was failing because it tried to copy `quant-server.exe` which CMakeLists.txt explicitly does not build on MSVC (`if(TQ_BUILD_SERVER AND NOT MSVC)` — uses POSIX sockets).

Changes:

  1. Windows package no longer references quant-server.exe
  2. Added `workflow_dispatch` so we can re-run release.yml for an existing tag without re-tagging
  3. Each `actions/checkout` step respects the tag input
  4. `softprops/action-gh-release` uses explicit `tag_name`

This unblocks v0.12.0 GitHub Release. PyPI v0.12.0 is already published.

🤖 Generated with Claude Code

The Windows build job tried to copy quant-server.exe which doesn't
exist — CMakeLists.txt:311 has `if(TQ_BUILD_SERVER AND NOT MSVC)` so
quant-server is intentionally not built on MSVC (uses POSIX sockets).

Changes:
1. Windows build no longer passes -DTQ_BUILD_SERVER=ON (no-op anyway)
   and the package step no longer references quant-server.exe.
2. Added workflow_dispatch trigger so the release workflow can be
   re-run for an existing tag without re-tagging.
3. Each checkout step now respects github.event.inputs.tag for
   workflow_dispatch runs.
4. softprops/action-gh-release uses explicit tag_name from input.

This unblocks the v0.12.0 GitHub Release (PyPI v0.12.0 already
published successfully).

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@unamedkr unamedkr merged commit 6db5943 into main Apr 11, 2026
@unamedkr unamedkr deleted the fix/release-windows-no-server branch April 11, 2026 06:12
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.

1 participant