approved

LC translation

Haskell translation

C++ Translation

C++ Translation. (Omits the reference solution, per the new JS fork)

Approved

LGTM! I would've personally loved to see generateTestCase further simplified as below, but it's good as-is

std::vector<int> result(randint(15, 100));
std::iota(result.begin(), result.end(), randint(-1e6, 1e6));

if (not isConsecutive) {
    const std::size_t index      = randint(1, result.size() - 2);
    const std::size_t eraseCount = randint(1, result.size() - 1 - index);
    result.erase(result.begin() + index, result.begin() + index + erase_count);
}

return { result, solution(result) };

Point 1 is now fixed too.

Point 2 is now fixed

• I heavily encourage you to add the Python fixed tests, splitting them into an It(all_consecutive) and It(non_consecutive). The two tests from JS are too little imo

• From the description:

  The numbers will also all be unique and in ascending order

  This means std::generate(result.begin(), result.end(), [&] { return randint(-1e6, 1e6); } ); is non-compliant since it can generate duplicates and is not guaranteed to be sorted

• Optioal, but fmt::format("numbers = {{{}}}\n", fmt::join(numbers, ", ")) is unnecessary because fmt::format("numbers = {}\n", numbers) will output the same exact thing

Thanks you for the translation. Approved.

I'm not sure that works. It might, but it might not.

Just fork all translations so they have the new description, and approve the forks. That definitely works.

Better to approve any translations you intend to approve, and only change the description in the editor ( or a translation! a translation can also change the description, and that also invalidates any other pending translations ) when none are pending.

If your reaction is "Huh? Wow.", yes, all of us think that.

Oh, sorry I didn't know about that. Should I revert the change, merge the translations, then add it back?