-
Notifications
You must be signed in to change notification settings - Fork 5.5k
Enhance Process.ReadAllLinesAsync #128151
Unanswered
NeverMorewd
asked this question in
General
-
Consider adding a new ReadAllLinesAsync overload that returns IObservable<ProcessOutputLine>.
Beta Was this translation helpful? Give feedback.
All reactions
Replies: 1 comment 2 replies
-
You can open an API proposal to discuss further. Note that the new method has to have a different name.
Beta Was this translation helpful? Give feedback.
All reactions
2 replies
-
Generally I don't expect it to be approved. The BCL doesn't provide any concrete implementation of IObservable, and Rx provides easy conversion from IEnumerable to IObservable.
Beta Was this translation helpful? Give feedback.
All reactions
-
Yes, I think it is more appropriate to implement it in Rx.
Beta Was this translation helpful? Give feedback.
All reactions
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment