search

rust-bakery / nom

Rust parser combinator framework
MIT License
9.18k stars 792 forks source link

implement alt for dynamic arrays #1754

Closed Geal closed 1 month ago

Geal commented 1 month ago

Fix #1720

codecov[bot] commented 1 month ago

Codecov Report

Attention: Patch coverage is 61.53846% with 5 lines in your changes are missing coverage. Please review.

Project coverage is 60.57%. Comparing base (82b09f0) to head (8ee8e8c). Report is 3 commits behind head on main.

Files Patch % Lines
src/branch/mod.rs 61.53% 5 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #1754 +/- ## ========================================== - Coverage 62.01% 60.57% -1.45% ========================================== Files 24 24 Lines 2917 3282 +365 ========================================== + Hits 1809 1988 +179 - Misses 1108 1294 +186 ```

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

codspeed-hq[bot] commented 1 month ago

CodSpeed Performance Report

Merging #1754 will not alter performance

Comparing alt-dynamic-array (8ee8e8c) with main (cea6595)

Summary

✅ 24 untouched benchmarks