search

pmem / pmemstream

Other
9 stars 13 forks source link

docs: add detailed description of internals into manpage .7 #250

Closed lukaszstolarczuk closed 2 years ago

lukaszstolarczuk commented 2 years ago

This change is Reviewable

codecov-commenter commented 2 years ago

Codecov Report

Merging #250 (857c575) into master (13aa782) will increase coverage by 0.13%. The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #250      +/-   ##
==========================================
+ Coverage   89.35%   89.48%   +0.13%     
==========================================
  Files           9        9              
  Lines         742      742              
  Branches      134      134              
==========================================
+ Hits          663      664       +1     
  Misses         47       47              
+ Partials       32       31       -1
Flag Coverage Δ
tests_gcc_debug_cpp17 89.48% <ø> (+0.13%) :arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
pmemstream/src/libpmemstream.c 88.16% <0.00%> (+0.25%) :arrow_up:

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more

lukaszstolarczuk commented 2 years ago

:lgtm: (you could also run it through Grammarly perhaps)

Reviewable status: 2 of 4 files reviewed, all discussions resolved (waiting on @igchor and @karczex)

I did, on the fly.