dotnet / dotnet/reactive

AsyncRx.NET's Do operator overloads more prone to ambiguity than normal Rx.NET'

Open
#2,170 0 comments 0 reactions 0 assignees View on GitHub
[area] AsyncRx
Dominant language
C#
Stars
7.2k
Forks
798
PR merge metrics
No merged PRs in 30d

Description

There are overloads of `AsyncEnumerable.Do` that accept an `Action`. The problem with this is that is can lead to ambiguity with the one that accepts an `Action`.

This has caused problems as we start to try to make the existing test suite for Rx.NET work on AsyncRx.NET.

Normal Rx.NET seems to avoid this by offering a more selective set of overloads. We should probably follow suit in AsyncRx.NET.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.