Skip to content

Conversation

@jbcoe
Copy link
Owner

@jbcoe jbcoe commented Oct 29, 2025

GCC-15 runs do not work (yet). It looks like GCC-15 is not installed on the test runners.

@jbcoe jbcoe changed the title Add CI targets for newer compilers Add CI targets for updated Clang and MSVC compilers Oct 30, 2025
@jbcoe jbcoe marked this pull request as ready for review October 30, 2025 00:24
@jbcoe jbcoe requested a review from Twon as a code owner October 30, 2025 00:24
},
lib: "libstdc++14",
}
# - {
Copy link
Collaborator

@Twon Twon Oct 31, 2025

Choose a reason for hiding this comment

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

Specifically, GCC-15 is not yet supported in the latest supported version of Ubuntu? Should we remove this until it's supported?

Copy link
Owner Author

@jbcoe jbcoe Oct 31, 2025

Choose a reason for hiding this comment

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

It's a reminder if left as is but we can do that with an issue. Probably best removed.

As you say, runners don't have the compiler version (yet).

Copy link
Owner Author

Choose a reason for hiding this comment

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

I'll remove the comments and leave this PR open until CI passes.

@jbcoe jbcoe requested a review from Copilot November 7, 2025 19:16
Copy link

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 adds support for newer compiler versions to the CI matrix and cleans up the workflow configuration by removing a nonexistent file from the ignore list.

Key Changes

  • Adds CI support for GCC-15, Clang-20, Visual Studio 2022 v20, and Apple Clang 16
  • Removes requirements.txt from clang-tidy workflow's paths-ignore list (file doesn't exist in repository)

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
.github/workflows/cmake.yml Adds four new compiler configurations to the build matrix: GCC-15, Clang-20, Visual Studio 2022 v20, and Apple Clang 16
.github/workflows/clang-tidy.yml Removes nonexistent requirements.txt from paths-ignore lists in both push and pull_request triggers

- {
name: "Visual Studio 2022 v20",
os: windows-latest,
compiler: { type: VISUAL, version: 20, cc: "cl", cxx: "cl", std: 20 },
Copy link

Copilot AI Nov 7, 2025

Choose a reason for hiding this comment

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

The std: 20 field is inconsistent with other Visual Studio entries (v16, v17, v18, v19) which don't include this field. For consistency with the existing pattern, either remove this field from the v20 entry or add it to all other Visual Studio entries. The field appears to not be used by the workflow steps, as the CMake configuration is handled by presets.

Suggested change
compiler: { type: VISUAL, version: 20, cc: "cl", cxx: "cl", std: 20 },
compiler: { type: VISUAL, version: 20, cc: "cl", cxx: "cl" },

Copilot uses AI. Check for mistakes.
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