-
Notifications
You must be signed in to change notification settings - Fork 177
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
ci: Add builds using C++20 #1957
Conversation
Codecov Report
@@ Coverage Diff @@
## main #1957 +/- ##
=======================================
Coverage 49.60% 49.60%
=======================================
Files 410 410
Lines 23297 23297
Branches 10630 10630
=======================================
Hits 11557 11557
Misses 4292 4292
Partials 7448 7448 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
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.
looks good to me 👍
📊 Physics performance monitoring for d8a3b71Full report VertexingSeedingCKFAmbiguity resolutionTruth tracking (Kalman Filter)Truth tracking (GSF) |
Is there actually a plan to migrate to C++20 anytime soon? |
Nothing concrete yet, but I thought it doesn't hurt to check that we're ready in principle. |
The main workflow continues to run on C++17, but this adds variants to the `extra` build matrix that builds in C++20, too. So we should now cover: - Ubuntu 20.04 + C++17 - Ubuntu 22.04 + C++17 and C++20 - Ubuntu 22.04 clang + C++17 and C++20
The main workflow continues to run on C++17, but this adds variants to the
extra
build matrix that builds in C++20, too.So we should now cover: