sideeffect-io / AsyncExtensions

AsyncExtensions aims to mimic Swift Combine operators for async sequences.
MIT License
333 stars 26 forks source link

project: fix onNext scheduling in ConcurrentAccessRegulator #11

Closed twittemb closed 2 years ago

twittemb commented 2 years ago

Description

This PR fixes a potential issue where the onNext function could be called in non deterministic order.

Checklist

codecov-commenter commented 2 years ago

Codecov Report

Merging #11 (6081b19) into main (aae817b) will decrease coverage by 0.00%. The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main      #11      +/-   ##
==========================================
- Coverage   97.32%   97.32%   -0.01%     
==========================================
  Files          44       44              
  Lines        2243     2240       -3     
==========================================
- Hits         2183     2180       -3     
  Misses         60       60              
Impacted Files Coverage Δ
Sources/Internal/ConcurrentAccessRegulator.swift 100.00% <100.00%> (ø)
...sts/AsyncSequences/AsyncSequences+MergeTests.swift 97.36% <100.00%> (-0.06%) :arrow_down:

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 aae817b...6081b19. Read the comment docs.