Expose Publisher.asAnyWorkflow as public API

[amorde]

Without this API consuming a Publisher inside a workflow is only possible by either rendering it directly via .render(in:) or by first calling mapOutput:

somePublisher
    .mapOutput { $0 } // workaround for not being able to call `asAnyWorkflow()`
    .onOutput { state, output in
         // ...
    }.rendered(in: context)

[amorde]

doing some cleanup while here - PublisherWorkflow isn't public so these don't need to be either

[mjohnson12]

Do we want to encourage using publishers directly in a workflow like this instead of using a worker?

[mjohnson12]

Do we want to encourage using publishers directly in a workflow like this instead of using a worker?

I guess we already indirectly exposed this with running and map output so I'm fine with it.

[amorde]

Do we want to encourage using publishers directly in a workflow like this instead of using a worker?

Is there a way to use PublisherWorkflow directly that I'm missing? Since its not public, the only way to use it is to call .render(in:) which skips other steps along the way

[mjohnson12]

Do we want to encourage using publishers directly in a workflow like this instead of using a worker?

Is there a way to use PublisherWorkflow directly that I'm missing? Since its not public, the only way to use it is to call .render(in:) which skips other steps along the way
You can do Publisher.running(in) which just calls asAnyWorkflow in the implementation so I'm fine with exposing asAnyWorkflow as public.

[amorde]

Oh I see - you mean manually defining your own worker like this.

The ReactiveSwift APIs allow you to use Signals directly similar to this PR, which is what prompted me to look for the same thing with Combine