search

Stranger6667 / jsonschema-rs

JSON Schema validation library
https://docs.rs/jsonschema
MIT License
511 stars 91 forks source link

fix: Combination of `unevaluatedProperties` with `allOf` and `oneOf` #509

Closed Stranger6667 closed 3 weeks ago

Stranger6667 commented 3 weeks ago

Fixes #496

codecov[bot] commented 3 weeks ago

Codecov Report

Attention: Patch coverage is 92.30769% with 3 lines in your changes missing coverage. Please review.

Project coverage is 84.70%. Comparing base (3c58526) to head (16549a7). Report is 1 commits behind head on master.

Files with missing lines Patch % Lines
.../jsonschema/src/keywords/unevaluated_properties.rs 92.30% 3 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #509 +/- ## ========================================== + Coverage 84.66% 84.70% +0.04% ========================================== Files 69 69 Lines 10910 10939 +29 ========================================== + Hits 9237 9266 +29 Misses 1673 1673 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

codspeed-hq[bot] commented 3 weeks ago

CodSpeed Performance Report

Merging #509 will degrade performances by 13.61%

Comparing dd/fix-unevaluated (16549a7) with master (3c58526)

Summary

❌ 1 (👁 1) regressions ✅ 16 untouched benchmarks

Benchmarks breakdown

Benchmark master dd/fix-unevaluated Change
👁 Fast/Invalid[is_valid] 740.3 ns 856.9 ns -13.61%