Change signature for `containsAll()`

[evant]

right now it's containsAll(vararg Any?) which it makes it too easy to accidentally pass in listOf(foo) instead of *listOf(foo)

[rf43]

Just curious about this... we could just have it throw an error (as it currently does). I could add another test to ensure that this behavior is kept and consistent.

Also, iirc - for now Kotlin only supports the spread operator for arrays *arrayOf(foo) and not collections like lists *listOf(foo)

[evant]

Hm good point, maybe a separate collection overload?

[evant]

Looks like this is impossible without https://youtrack.jetbrains.com/issue/KT-13198

[nishtahir]

Looks like a blocker. Can't think of a way around this 😕

[GrahamLea]

containsAll(*listOf(...)) doesn't event work for me. Need to put .toTypedArray() on the end too. Deal-breaker. :(

[jzbrooks]

Looks like this is impossible without https://youtrack.jetbrains.com/issue/KT-13198

@evant, could you elaborate on this? I added an Iterable<T> overload here that seems to work okay. It seems like some of the rough corners in generics have been improved in K2 (so maybe even if this doesn't work it will be addressed by upgrading to K2)

main...jzbrooks:assertk:iterable-overload