Skip to content

Pure GAP fallback for JSON parser - #39

Draft
fingolfin wants to merge 2 commits into
masterfrom
pure-gap-fallback
Draft

Pure GAP fallback for JSON parser#39
fingolfin wants to merge 2 commits into
masterfrom
pure-gap-fallback

Test both implementations, and both against JSONTestSuite

bd5c9a0
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project succeeded Aug 18, 2026 in 0s

97.59% (+1.16%) compared to 68c1ba7

View this Pull Request on Codecov

97.59% (+1.16%) compared to 68c1ba7

Details

Codecov Report

❌ Patch coverage is 97.60589% with 13 lines in your changes missing coverage. Please review.
✅ Project coverage is 97.59%. Comparing base (68c1ba7) to head (bd5c9a0).

Files with missing lines Patch % Lines
gap/parse.gi 96.85% 12 Missing ⚠️
gap/utf8.gi 99.05% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master      #39      +/-   ##
==========================================
+ Coverage   96.42%   97.59%   +1.16%     
==========================================
  Files           6       10       +4     
  Lines         140      791     +651     
==========================================
+ Hits          135      772     +637     
- Misses          5       19      +14     

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