No random tests.

To avoid copying of a std::vector

What's the reason for changing the function signature?

Hi @skaarj1989, I think this translation could be improved by adding some random test cases.

Merge conflict (please fork the description)

Approved

Fixed, thanks for info.

For low level kata I recommend to print all input values (using it block titles). See this kata for reference.

As of now CW uses Clang8 as the C++ compiler, and you have to write code which would work correctly with Clang8 regardless of what tools you're using locally. Also, it looks like you're trying to sort the array in reverse order - if my guess is correct, then it seems that MSVC is wrong because "z" > "ya". Either way not a kata issue.

std::sort gives me different results for MSVC and Clang 8
Just because of this I have a failed test (Edge_Cases)

Expected: equal to "ya" Actual: "z"

Clang8 = [z ya x w v u t s r q p o n m l k j i h g f e d c b a]
MSVC = [ya z x w v u t s r q p o n m l k j i h g f e d c b a]