Skip to content

Add post-merge CI jobs for dev versions of dependencies#1962

Merged
bryanwweber merged 7 commits intoCantera:mainfrom
speth:dev-dependency-tests
Sep 3, 2025
Merged

Add post-merge CI jobs for dev versions of dependencies#1962
bryanwweber merged 7 commits intoCantera:mainfrom
speth:dev-dependency-tests

Conversation

@speth
Copy link
Copy Markdown
Member

@speth speth commented Sep 1, 2025

Changes proposed in this pull request

  • Add post-merge CI jobs using (separately) the latest development versions of Eigen, SUNDIALS, libfmt, yaml-cpp, HighFive, and SCons
  • Add post-merge CI jobs using Docker images for Debian stable, testing, and unstable images
  • Update post-merge CI job using Homebrew Python to also use Homebrew-provided GCC to compile Cantera
  • Fix handling of the name of the HDF5 library when using a "system" version of HighFive (can be hdf5_serial, as we automatically detect)
  • Fix setting of RPATH flags when using Homebrew GCC
  • Fix the error message provided when the system_sundials=y flag is specified but SUNDIALS is not found

If applicable, fill in the issue number this pull request is fixing

If applicable, provide an example illustrating new features this pull request is introducing

Checklist

  • The pull request includes a clear description of this code change
  • Commit messages have short titles and reference relevant issues
  • Build passes (scons build & scons test) and unit tests address code coverage
  • Style & formatting of contributed code follows contributing guidelines
  • The pull request is ready for review

@speth speth added the CI label Sep 1, 2025
@speth speth force-pushed the dev-dependency-tests branch 3 times, most recently from 84310ed to 4136633 Compare September 1, 2025 20:54
@codecov
Copy link
Copy Markdown

codecov bot commented Sep 1, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 75.04%. Comparing base (38fbee0) to head (11a1078).
⚠️ Report is 19 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1962      +/-   ##
==========================================
+ Coverage   74.99%   75.04%   +0.05%     
==========================================
  Files         450      450              
  Lines       56235    56228       -7     
  Branches     9300     9297       -3     
==========================================
+ Hits        42173    42199      +26     
+ Misses      10930    10899      -31     
+ Partials     3132     3130       -2     

☔ 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.

@speth speth force-pushed the dev-dependency-tests branch 3 times, most recently from a63b657 to 3760a40 Compare September 2, 2025 20:56
@speth speth force-pushed the dev-dependency-tests branch 7 times, most recently from 6ad6fc7 to d7b4eb2 Compare September 3, 2025 03:53
@speth speth force-pushed the dev-dependency-tests branch from d7b4eb2 to 11a1078 Compare September 3, 2025 14:14
@speth speth marked this pull request as ready for review September 3, 2025 15:49
Copy link
Copy Markdown
Member

@bryanwweber bryanwweber left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks Ray!

@bryanwweber bryanwweber merged commit e9fa595 into Cantera:main Sep 3, 2025
68 of 70 checks passed
@speth speth deleted the dev-dependency-tests branch September 3, 2025 23:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants