Skip to content

Refactor validator return type to support warnings via ValidationResult wrapper (#1479) - #1480

Open
sejalpunwatkar wants to merge 19 commits into
hdmf-dev:devfrom
sejalpunwatkar:feature/1479-validation-result
Open

Refactor validator return type to support warnings via ValidationResult wrapper (#1479)#1480
sejalpunwatkar wants to merge 19 commits into
hdmf-dev:devfrom
sejalpunwatkar:feature/1479-validation-result

Merge branch 'feature/1479-validation-result' of https://github.com/s…

6ded0be
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project failed Aug 2, 2026 in 1s

37.96% (-55.19%) compared to 4a8967f

View this Pull Request on Codecov

37.96% (-55.19%) compared to 4a8967f

Details

Codecov Report

❌ Patch coverage is 60.00000% with 10 lines in your changes missing coverage. Please review.
✅ Project coverage is 37.96%. Comparing base (4a8967f) to head (6ded0be).

Files with missing lines Patch % Lines
src/hdmf/validate/errors.py 61.90% 8 Missing ⚠️
src/hdmf/validate/validator.py 33.33% 2 Missing ⚠️

❗ There is a different number of reports uploaded between BASE (4a8967f) and HEAD (6ded0be). Click for more details.

HEAD has 4 uploads less than BASE
Flag BASE (4a8967f) HEAD (6ded0be)
5 1
Additional details and impacted files
@@             Coverage Diff             @@
##              dev    #1480       +/-   ##
===========================================
- Coverage   93.15%   37.96%   -55.19%     
===========================================
  Files          41       41               
  Lines       10245    10264       +19     
  Branches     2119     2119               
===========================================
- Hits         9544     3897     -5647     
- Misses        422     6001     +5579     
- Partials      279      366       +87     

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