Skip to content

Update CI#214

Merged
ekilmer merged 5 commits into
masterfrom
update-windows-ci
May 12, 2026
Merged

Update CI#214
ekilmer merged 5 commits into
masterfrom
update-windows-ci

Conversation

@ekilmer
Copy link
Copy Markdown
Contributor

@ekilmer ekilmer commented May 12, 2026

  • Pin GitHub Actions using SHA (pinact run --update)
  • Bump Windows runner and Visual Studio CMake Generator
  • Remove std::filesystem workaround. It's 2026 now
  • Update to latest Python versions for testing
  • Use latest gh-action-pypi-publish workflow guidance (not tested... PYPI project is archived)
  • Remove unmaintained ilammy/msvc-dev-cmd action

@ekilmer ekilmer force-pushed the update-windows-ci branch from 7203fe3 to bf5a786 Compare May 12, 2026 17:27
MSVC links /fsanitize=address binaries against
clang_rt.asan_dynamic-x86_64.dll, which only sits on PATH inside a VS
developer prompt. ctest launched the test binaries in a bare shell, so
they failed at startup with STATUS_DLL_NOT_FOUND (0xc0000135).

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@ekilmer ekilmer force-pushed the update-windows-ci branch from bf5a786 to be529f7 Compare May 12, 2026 17:37
@ekilmer ekilmer marked this pull request as ready for review May 12, 2026 18:11
@ekilmer ekilmer mentioned this pull request May 12, 2026
@gsutherland-trailofbits
Copy link
Copy Markdown
Collaborator

I think this one is good. The referenced PR, #211, will probably need some changes once this is merged, as it also changes the CI runner VS version (to an older version than this PR does). But that's a relatively minor thing, and the rest of that PR looks good.

Can you squash/merge, or do you need me to boop the button?

@ekilmer ekilmer merged commit 1bb8db5 into master May 12, 2026
40 checks passed
@ekilmer ekilmer deleted the update-windows-ci branch May 12, 2026 22:52
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