Closed jameslittle230 closed 2 years ago
Merging #270 (d814f1f) into master (eeaca67) will increase coverage by
0.46%
. The diff coverage is75.00%
.:exclamation: Current head d814f1f differs from pull request most recent head e47d212. Consider uploading reports for the commit e47d212 to get more accurate results
@@ Coverage Diff @@
## master #270 +/- ##
==========================================
+ Coverage 72.44% 72.91% +0.46%
==========================================
Files 53 52 -1
Lines 2174 2160 -14
Branches 104 104
==========================================
Hits 1575 1575
+ Misses 598 584 -14
Partials 1 1
Impacted Files | Coverage Δ | |
---|---|---|
stork-cli/src/clap.rs | 100.00% <ø> (ø) |
|
stork-cli/src/main.rs | 0.00% <0.00%> (ø) |
|
stork-lib/src/index_v3/build/fill_stems.rs | 100.00% <ø> (ø) |
|
stork-lib/src/index_v3/mod.rs | 70.37% <ø> (ø) |
|
stork-lib/src/index_v3/build/fill_containers.rs | 91.80% <100.00%> (ø) |
|
stork-lib/src/index_v3/build/mod.rs | 98.57% <100.00%> (ø) |
|
stork-lib/src/index_v3/search/mod.rs | 95.77% <100.00%> (ø) |
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact)
,ø = not affected
,? = missing data
Powered by Codecov. Last update d2ed81c...e47d212. Read the comment docs.
Benchmark | Baseline | Contender | Comparison |
---|---|---|---|
build/federalist | 167.6459 | 166.055 | 0.99× 🎉 |
federalist.st | 1125.456 | 1125.456 | 1.0× |
search/federalist/liberty | 1.4581 | 1.4545 | 1.0× |
stork.js | 21.88 | 21.88 | 1.0× |
stork.wasm | 356.523 | 356.523 | 1.0× |
Baseline: d2ed81c8a379982f7290c625c0440b838488f573; Comparison: e47d2124766bb215dbce3dbb4abc945d1deb4d12
Problems:
feature
that didn't exist, so it was never being compiled inindex_path
command line argument wasn't being extracted properly since I was querying it with the incorrect stringindex
Fixes #269