Skip to content

Conversation

@nikobockerman
Copy link
Owner

No description provided.

Copilot AI review requested due to automatic review settings January 10, 2026 19:54
Copy link
Contributor

Copilot AI left a comment

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 migrates the primary CI build matrix from macOS to Ubuntu while retaining a single macOS build configuration for release testing. The change reduces the macOS build coverage from multiple compiler/library combinations to just one (clang with libstdc++ in release mode), while expanding Ubuntu builds to cover the full matrix of compiler and library configurations.

Changes:

  • Changed the primary build matrix OS from macOS to Ubuntu
  • Restructured the workflow matrix to include Ubuntu builds with both clang and gcc compilers
  • Updated CMake presets to reflect the new OS priorities, removing macOS-specific debug and gcc configurations

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
.github/workflows/cpp-build-test-run.yaml Modified build matrix to use Ubuntu as primary OS and adjusted the include matrix to swap OS-specific configurations
.github/files/CMakeUserPresets.json Reorganized and renamed CMake presets to prioritize Ubuntu configurations, removing most macOS preset variants

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

@nikobockerman nikobockerman force-pushed the cpp-focus-on-ubuntu-runner branch from 7028e13 to 1dc95e3 Compare January 10, 2026 20:27
Copilot AI review requested due to automatic review settings January 11, 2026 14:43
@nikobockerman nikobockerman force-pushed the cpp-focus-on-ubuntu-runner branch from 1dc95e3 to 9ee7c7b Compare January 11, 2026 14:43
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated no new comments.


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

@nikobockerman nikobockerman force-pushed the cpp-focus-on-ubuntu-runner branch 2 times, most recently from 31d8d16 to 0463119 Compare January 11, 2026 15:45
@nikobockerman nikobockerman enabled auto-merge (squash) January 11, 2026 15:46
Copilot AI review requested due to automatic review settings January 11, 2026 15:53
@nikobockerman nikobockerman force-pushed the cpp-focus-on-ubuntu-runner branch from 0463119 to 462b9e9 Compare January 11, 2026 15:53
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated no new comments.


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

@nikobockerman nikobockerman force-pushed the cpp-focus-on-ubuntu-runner branch from 462b9e9 to 94acd8c Compare January 11, 2026 16:22
@nikobockerman nikobockerman merged commit aade367 into main Jan 11, 2026
46 checks passed
@nikobockerman nikobockerman deleted the cpp-focus-on-ubuntu-runner branch January 11, 2026 16:43
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