bionic icon indicating copy to clipboard operation
bionic copied to clipboard

`FlowBuilder.merge` not accepting another builder is confusing

Open jqmp opened this issue 5 years ago • 0 comments
trafficstars

The FlowBuilder.merge method excepts a flow, not a builder, which seems to be a common source of surprise. What's worse, if you pass it a builder you get a cryptic error message about not having a name attribute.

We should either check the type and raise a clearer exception, or just accept a builder as well.

jqmp avatar Aug 20 '20 00:08 jqmp