spock icon indicating copy to clipboard operation
spock copied to clipboard

Add documentation for concurrency helpers

Open thokari opened this issue 4 years ago • 2 comments

This is for #1058 , please let me know if this is what you expect, then I will wrap it up for the other classes. Also the indentation looks a bit strange for the number hints on the AgeFilter example, I'd fix that as well if you don't mind.


This change is Reviewable

thokari avatar May 31 '20 15:05 thokari

Codecov Report

Merging #1173 (05e6d3e) into master (b8bfe54) will not change coverage. The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master    #1173   +/-   ##
=========================================
  Coverage     76.52%   76.52%           
- Complexity     3754     3756    +2     
=========================================
  Files           405      405           
  Lines         11437    11437           
  Branches       1396     1395    -1     
=========================================
  Hits           8752     8752           
  Misses         2197     2197           
  Partials        488      488           

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 b8bfe54...05e6d3e. Read the comment docs.

codecov[bot] avatar May 31 '20 16:05 codecov[bot]

@leonard84 , @Vampire , @thokari Will this ever be merged? Testing concurrency is always hard, insufficient documentation makes it even harder.

winne42 avatar Apr 25 '23 12:04 winne42