broadcastIn's ReplaceWith is incorrect

[ansman]

I tried using the ReplaceWith for broadcastIn in 1.4.0 and it has several issues.

Firstly it doesn't handle CoroutineStart other than lazily which seems a bit dangerous as it might change behavior.

Secondly the order of the parameters is incorrect. In the ReplaceWith it's listed as shareIn(scope, 0, SharingStarted.Lazily) while the final version of shareIn has scope, started, replay.

Lastly it was just broken:

someFlow
  .broadcastIn(viewModelScope)
// became
run {
  flow
  shareIn<Type>(viewModelScope, 0, SharingStarted.Lazily)
}

One solution would be to add a deprecated, single argument, overload for broadcastIn which is replaced with shareIn(scope) and the 2 argument overload can have its ReplaceWith removed since it cannot be done reliably.