-
Notifications
You must be signed in to change notification settings - Fork 15
Add CI targets for updated Clang and MSVC compilers #603
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
.github/workflows/cmake.yml
Outdated
| }, | ||
| lib: "libstdc++14", | ||
| } | ||
| # - { |
There was a problem hiding this comment.
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?
There was a problem hiding this comment.
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).
There was a problem hiding this comment.
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.
…bcoe/value_types into dev-jbcoe-github-ci-compiler-additions
There was a problem hiding this 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.txtfrom 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 }, |
Copilot
AI
Nov 7, 2025
There was a problem hiding this comment.
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.
| compiler: { type: VISUAL, version: 20, cc: "cl", cxx: "cl", std: 20 }, | |
| compiler: { type: VISUAL, version: 20, cc: "cl", cxx: "cl" }, |
GCC-15 runs do not work (yet). It looks like GCC-15 is not installed on the test runners.