codecov-commenter
commented
1 year ago Codecov Report
Merging #77 (7882db1) into main-with-3.4.0 (7ea0b02) will increase coverage by
4.89%. The diff coverage is94.84%.
@@ Coverage Diff @@
## main-with-3.4.0 #77 +/- ##
===================================================
+ Coverage 90.08% 94.97% +4.89%
===================================================
Files 10 10
Lines 625 637 +12
===================================================
+ Hits 563 605 +42
+ Misses 62 32 -30 | Files Changed | Coverage Δ | |
|---|---|---|
| lib/RDFC10Sync.js | 97.63% <88.88%> (ø) |
|
| lib/index.js | 63.79% <93.75%> (+5.30%) |
:arrow_up: |
| lib/MessageDigest-webcrypto.js | 84.61% <100.00%> (+1.28%) |
:arrow_up: |
| lib/MessageDigest.js | 100.00% <100.00%> (ø) |
|
| lib/NQuads.js | 98.19% <100.00%> (+15.99%) |
:arrow_up: |
| lib/RDFC10.js | 98.85% <100.00%> (ø) |
Continue to review full report in Codecov by Sentry.
Legend - Click here to learn more
Δ = absolute <relative> (impact),ø = not affected,? = missing dataPowered by Codecov. Last update 7ea0b02...7882db1. Read the comment docs.
AbortSignalsignaloption for complexity control via timeouts, manual abort, or other condition.timeoutto private sync version._:prefix inBlankNodesto improve compatibility with RDF/JS tooling. (Need to test what other tooling issues this may cause, for instance in jsonld.js.)A 1000ms default timeout is set here. I'm not sure if that's the best approach or have no abort signal here and let other libraries and apps add a default that fits their use case.