Skip to content

add CI workflow for MSVC and fix GDI demo#972

Open
sleeptightAnsiC wants to merge 2 commits into
Immediate-Mode-UI:masterfrom
sleeptightAnsiC:ci_add_msvc
Open

add CI workflow for MSVC and fix GDI demo#972
sleeptightAnsiC wants to merge 2 commits into
Immediate-Mode-UI:masterfrom
sleeptightAnsiC:ci_add_msvc

Conversation

@sleeptightAnsiC

@sleeptightAnsiC sleeptightAnsiC commented Jun 30, 2026

Copy link
Copy Markdown
Contributor

Notes:

  • setting up Windows/MSVC manually would be probably a huge headache. I just reused ilammy/msvc-dev-cmd@v1 action which seems to be the most popular one.
  • every demo/*/build.bat hardcodes directory to the same VC2015 toolchain. That's why CI uses arch: x86 and toolset: 14.0. Would be nice to clean it up someday but for now I just want to get it in
  • the workflow is in separate file. I thought it makes sense since it needs to run under entirely different OS/toolchain and builds entirely different demos
  • GDI demo appears to be broken since 0639685. I've patched it in separate commit.
  • build d3d11 step is noticeably slower. I don't have any clear explanation for this. Maybe because it is first to run, maybe because of different shader compilation, I'm not sure...
  • there are quite a lot of warnings. I may fix those later as part of Compiler warnings #958
  • I'm a total noob when it comes to Github CI, so maybe there is something obvious I didn't notice
  • DO NOT SQUASH

Refs: #895
Refs: #937

This file was failing with syntax error which appears to be broken since
Immediate-Mode-UI@0639685
I suspect none of the Windows-specific demos were actually tested.

Refs: Immediate-Mode-UI#937
Adds CI workflow for all Windows-specific demos that require MSVC.
Because each build.bat hardcodes path to the same MSVC toolchain,
the workflow uses arch:x86 and toolset:14.0 for compatibility.

Refs: Immediate-Mode-UI#895
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