Skip to content

Property definitions ci fixes - #2449

Open
rartino wants to merge 2 commits into
Materials-Consortia:ml-evs/property-definitionsfrom
rartino:property-definitions-ci-fixes
Open

rartino wants to merge 2 commits into
Materials-Consortia:ml-evs/property-definitionsfrom
rartino:property-definitions-ci-fixes

Conversation

@rartino

@rartino rartino commented Oct 31, 2026 •

Copy link
Copy Markdown
Member

With some help from Claude, I think these should fix the CI failures in #2442

Note that this PR is a PR against the branch of #2442

@codecov

codecov Bot commented Oct 31, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 90.51%. Comparing base (160f9b6) to head (b17ecea).

Additional details and impacted files
@@                       Coverage Diff                       @@
##           ml-evs/property-definitions    #2449      +/-   ##
===============================================================
- Coverage                        90.56%   90.51%   -0.06%     
===============================================================
  Files                               79       79              
  Lines                             5364     5364              
===============================================================
- Hits                              4858     4855       -3     
- Misses                             506      509       +3     
Flag Coverage Δ
project 90.51% <100.00%> (-0.06%) ⬇️
validator 90.51% <100.00%> (-0.06%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

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

This branch has not been deployed

No deployments
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