bartelink
commented
2 years ago In this specific instance (which also applied to Propulsion, which cloned this impl), there are some minor variations that I've had @eiriktsarpalis confirm are not useful additions:
- removal of redundant message text
- removal of guard for theoretical unknown state
- mapping cancellation to an exception (i.e., calling
ecvsccwhentask.IsCancelled)
While we await https://github.com/fsharp/fslang-suggestions/issues/840, this PR canonicalizes the implementation here to align with the Equinox and fsssnip edition in order to avoid people having to eliminate variation in the implementation from their inquiries when investigating threading issues.