Skip to content

Conversation

@DeeDeeG
Copy link
Member

@DeeDeeG DeeDeeG commented Oct 17, 2025

See commit messages for details.

Makes CI pass.

CI: Update from windows-2019 to windows-latest

windows-2019 images are discontinued, switch to '-latest'.


CI: Update node-gyp on Windows Node v14 jobs

Need newer node-gyp to support newer Visual Studio versions.

(Node-gyp 9 is the newest node-gyp that is compatible with NodeJS 14. It's new enough to support the Visual Studio version used in CI, though.)


CI: Pin macOS runner image to macos-14

macos-latest resolves to macos-15, which is too new to work with one of our dependencies. Specifically: there are compilation errors in git-utils --> libgit2 --> zlib when compiling with too new of an Xcode toolchain, apparently.

So, pin to the macos-14 image, which still works with our dependencies.

windows-2019 images are discontinued, switch to '-latest'.
Need newer node-gyp to support newer Visual Studio versions.

(Node-gyp 9 is the newest node-gyp that is compatible with NodeJS 14.
It's new enough to support the Visual Studio version used in CI,
though.)
macos-latest resolves to macos-15, which is too new to work
with one of our dependencies. Specifically: there are
compilation errors in git-utils --> libgit2 --> zlib
when compiling with too new of an Xcode toolchain, apparently.

So, pin to the macos-14 image, which still works with our dependencies.
Copy link
Contributor

@savetheclocktower savetheclocktower left a comment

Choose a reason for hiding this comment

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

Looks like CI to me. Looks good to me!

@DeeDeeG
Copy link
Member Author

DeeDeeG commented Oct 18, 2025

Thanx 🫡

It's mergin' time

@DeeDeeG DeeDeeG merged commit 79a60ea into master Oct 18, 2025
11 checks passed
@DeeDeeG DeeDeeG deleted the fix-CI-2025 branch October 18, 2025 03:41
@DeeDeeG DeeDeeG mentioned this pull request Nov 5, 2025
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.

3 participants