-
Notifications
You must be signed in to change notification settings - Fork 36
Add C23 Kernel #379
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
Add C23 Kernel #379
Conversation
Thanks for opening this. Saying this cause I've gotten multiple messages on mail and LinkedIn for a C kernel (yes only C and not c++) |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #379 +/- ##
=======================================
Coverage 81.78% 81.78%
=======================================
Files 20 20
Lines 950 950
Branches 87 87
=======================================
Hits 777 777
Misses 173 173 🚀 New features to boost your workflow:
|
Feels like since we have a c++17 kernel, then it might make sense to add a c17 standard kernel as well, but this is just my opinion. |
@davy39, thanks for your pull request. Do you think you can add kernels for the other standard c versions, too? |
Now my PR include c11, c17 and c23. |
Thank you! @mcbarton do you know why the docs build fails? |
@vgvassilev Honestly I have no idea. It looks like a random failure. It doesn't appear to be anything to do with main as I rebased the sub interpreter PR #380 to check and it built the read the docs. My suggestion is for @davy39 to either squash his 2 commits and repush, or push an empty commit to rerun the read the docs workflow. |
eb040d9
to
25181d0
Compare
This appears to be passing the read the docs workflow now. I will take this in once the ci goes green. |
Sorry for messing up with git... I had to force-push few times to get what I wanted :) |
Description
Add a new Clang C23 kernel, for those who need to code in C and not with C++ (lot of students in France).
Edit : + c11 & c17
Type of change
Please tick all options which are relevant.