Skip to content

Update CI builds#254

Merged
tcbrindle merged 14 commits intomainfrom
pr/clang-21-ci
Oct 8, 2025
Merged

Update CI builds#254
tcbrindle merged 14 commits intomainfrom
pr/clang-21-ci

Conversation

@tcbrindle
Copy link
Copy Markdown
Owner

No description provided.

@codecov
Copy link
Copy Markdown

codecov bot commented Oct 7, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 97.74%. Comparing base (6f51634) to head (38bb053).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #254      +/-   ##
==========================================
- Coverage   98.29%   97.74%   -0.55%     
==========================================
  Files          69      146      +77     
  Lines        2632     5149    +2517     
  Branches        0      244     +244     
==========================================
+ Hits         2587     5033    +2446     
- Misses         45       80      +35     
- Partials        0       36      +36     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Maybe I should have made sure things compile warning-free first...

(I'm pretty sure this is a just bogus warning from Clang 16 that was corrected in later compiler versions.)
@tcbrindle tcbrindle changed the title Add Clang 21 to CI Update CI builds Oct 7, 2025
A new GCC warning picked this up (good!), but I'm not sure how it is I haven't noticed it till now...
* Use GCC15
* Fix warnings
* Update git-auto-commit-action to v6
* Use GCC 15
* Set compiler/linker flags on the CMake command line rather than using Flux's CMake option (which will shortly be removed)
* Actually test both release and debug configs rather than just pretending to
@tcbrindle tcbrindle force-pushed the pr/clang-21-ci branch 2 times, most recently from 38bb053 to 42b8270 Compare October 8, 2025 10:37
@tcbrindle tcbrindle merged commit e8936e5 into main Oct 8, 2025
124 of 127 checks passed
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.

1 participant