Ad
  • Custom User Avatar

    python new test framework is required. updated in this fork

  • Custom User Avatar

    Why do we have 4 cells in this kata? 3 cells is enough for the task.

  • Custom User Avatar

    Ruby 3.0 should be enabled, see this to learn how to do it

    Please organize the structure of test fixture as following (although it has been mentioned in the attached link, I'm repeated here again)

    describe "<This message should describe the categories of test groups inside this block>" do
      it "<A short message describing this test group>" do
        expect(...).to eq(...) #Assertions
        #Or Test.assert_equals(user_response, reference_response)
      end
    end
    
  • Custom User Avatar
    • This is incorrect:

    . will return an empty string if no output was made in the code.

    Rather:

    your output will be considered as an empty string if no . command is executed at runtime.

    • putting information is the discourse with the spoiler flag makes that info invisible to people who need it, since they didn't solve the kata yet. They can post to get the info, yeah, but it's a bit cumbersome... Are you sure the flag is needed? Couldn't you reword some parts so that it's not needed anymore?

    • (and you can remove the "read plz" part: the related people won't care at all about that...)

  • Custom User Avatar
    • The description should explain the task properly, even if your kata is a performance-oriented version of another kata
    • No small random tests
    • "string too large to print" error messages are useless
    • The input generation is terribly slow
    • New test framework should be used
    • There're already numerous O(n) solutions to the original kata, and you can't stop anybody from copy-pasting those