• Custom User Avatar

    Python: In the description it says: He told you that the sub function may or may not be here, if no sub-function return undefined, ...

    The undefined part of sentence should be replaced by e.g. an empty string in Python

  • Custom User Avatar

    The description should probably state that the divisors are non-zero.

  • Custom User Avatar

    Tests don't even run correctly.

  • Custom User Avatar

    python new test framework is required. updated in this fork

  • Custom User Avatar

    Even though an issue on this subject (changing the name of a parameter from compare to key) was marked as resolved, I feel the Python translation needs a bit more work. The description still contains Javascript code which includes the line res = linkedSort(HowMany,Type,function(a,b){return a-b;}). This still misled me into thinking the key parameter would in fact be a two-parameter compare-style sorting function, (which I would need to use cmp_to_key on before using it with any built-in Python sorting functions).

    It does not help that neither of the example tests have a custom sorting function, so there are no code examples to help demonstrate what sort of functions will be passed in.

  • Custom User Avatar

    JS: Not enough fixed test cases. I have to guess the method signature of the comparison function.

  • Custom User Avatar

    Ruby 3.0 should be enabled.

  • Custom User Avatar

    Ruby 3.0 should be enabled.

  • Custom User Avatar

    The class name should be in PascalCase.

  • Custom User Avatar

    It's not mentioned that gueZZ may not receive an argument at all.

  • Custom User Avatar

    This solution incorrectly checks for subfunction but still passes the tests.

  • Custom User Avatar

    if no sub-function return undefined

    This does not apply to all kata versions.

  • Custom User Avatar
    • lacking a valid test with DEBUG
    • need random tests
  • Custom User Avatar

    This comment is hidden because it contains spoiler information about the solution

  • Custom User Avatar

    This comment is hidden because it contains spoiler information about the solution

  • Loading more items...