Skip to content

Add tests-passed job to ci - #86

Merged
mcflugen merged 1 commit into
csdms:masterfrom
mcflugen:add-tests-passed-ci-job
Oct 19, 2026
Merged

mcflugen merged 1 commit into
csdms:masterfrom
mcflugen:add-tests-passed-ci-job

Conversation

@mcflugen

Copy link
Copy Markdown
Member

I've added a new job, tests-passed, to the ci. All it does is check that the entire test matrix has passed.

This allows branch protection on GitHub to require just a single check rather than specifying every item of the build matrix as a required check, which is difficult to maintain as we add and remove Python versions. I've done this with the bmi-tester, and it seems to work pretty well.

@mcflugen
mcflugen force-pushed the add-tests-passed-ci-job branch from 8b930a7 to 8418115 Compare October 19, 2026 01:31
@codecov-commenter

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 96.25%. Comparing base (03e8e44) to head (8418115).

Additional details and impacted files
@@           Coverage Diff           @@
##           master      #86   +/-   ##
=======================================
  Coverage   96.25%   96.25%           
=======================================
  Files           8        8           
  Lines         267      267           
  Branches       11       11           
=======================================
  Hits          257      257           
  Misses          8        8           
  Partials        2        2           

☔ View full report in Codecov by Harness.
📢 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.

@mcflugen
mcflugen merged commit f6e53b9 into csdms:master Oct 19, 2026
21 checks passed
@mcflugen
mcflugen deleted the add-tests-passed-ci-job branch October 19, 2026 01:41
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.

2 participants