-
Notifications
You must be signed in to change notification settings - Fork 6
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
Update ci.yml and CMakePresets to enable gcc-{12, 13} and clang-19 #20
Update ci.yml and CMakePresets to enable gcc-{12, 13} and clang-19 #20
Conversation
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.
Welcome to Beman and congratulations for your first contribution!
0d0847f
to
0833f67
Compare
Signed-off-by: Radu Nichita <[email protected]>
Signed-off-by: Radu Nichita <[email protected]>
0833f67
to
b1d3267
Compare
@@ -1,75 +1,42 @@ | |||
name: Test | |||
|
|||
name: CI Tests |
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.
Can you confirm if this is exactly the same config from optional26?
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.
updated description
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.
LGTM 👍
I also tested all preset combinations on my local Ubuntu.
clang-{17, 18, 19} and gcc-{12, 13, 14} work on local + CI setup.
Thanks again @RaduNichita and welcome to our community! Looking forward for other contributions!
The PR imports the
ci.yml
andCMakePresets
from optional26 repository:Enable ASAN run with Clang-19 on CI
Enable gcc-14
Enable gcc-13
Enable gcc-12
Enable clang-19
Enable clang-18
Enable clang-17
Partially Fixes #18 and partially fixes #17