Ad
  • Default User Avatar

    Hi @Blind4Basics, may I know the exact reason the kata was retired? Just so that I can improve next time. Was it the poor tests, was the kata concept itself? Apart from that, was'nt the kata retired too fast? I mean, is the first kata I try to release, but I tried my best to be working on its fixes and I was thinking about refactoring the entire tests this weekend, free of college, but not even 48 hours past before it was retired. I feel like I needed more feedback about how much time I had to fix it + what can I improve.

    Thank you.

  • Default User Avatar

    Hey, I was sometimes checking those cases on the random cases but not always were generated since it needed to highroll lol. I have added fixed cases to make it functional by now but I will refactor the entire random tests generation to make sure it works there too 100% of the times.

    Thank you!

  • Default User Avatar

    Random cases added too.

  • Default User Avatar

    Hi again, I get this message when I try to delete my kata: 'User does not have permission to perform this action'.

  • Default User Avatar

    Makes sense, I have added fixed ones. I will be adding random ones of those too asap. Thank you so much for the feedback, it's my first time creating a kata or tests in general (outside of very simple college activities) an this is truly helping me understand what good tests are. ^^

  • Default User Avatar

    I'm trying to understand your solution and I think it assumes the given string 'nif' is always going to be 8 numbers and 1 letter, which was correct because the description I wrote in the first place said that, my bad. Now I have changed the description and some test cases so that you have to validate the length. I added cases where letters are inserted between the digits. Also, added cases with lowercase letters.
    Maybe I'm wrong again.

    I did'nt upvote your solution but I will upvote your fork since I learned the unpackage thingy which seems so useful :)

  • Default User Avatar

    Hi @Blinf4Basics, I added those extra cases. Thank you!

  • Default User Avatar

    You are right, I removed 'of exactly 8 digits and 1 letter' from the sentence. Thank you.

  • Default User Avatar

    I'm trying to update the "Initial solution" field on the kata but the changes do not apply when previewing or releasing the kata. It stays if I leave and join again the edit view however. And yes, I'm clicking save button. Am I missing something?

  • Default User Avatar

    My bad, samples should be added now, thank you.

  • Default User Avatar

    Hey, I'm using String in my solution, I just thought it would look more clear on description, bad decision. I will fix it so that it looks more clear. Thank you for the feedback.

  • Default User Avatar

    Hey, I used the word 'list' to refer to list as a concept, not the python keyword, I don't even use tuple or list in my solution, but instead an string version of it. I thought it would look more clear if I separated the chars with comas and parenthesis on the markdown, bad decision.
    The 'T' and the comma was a mistake. I will fix all things, thank you for the feedback.

  • Default User Avatar

    Hey, yeah sorry I just wanted to test the thingy, is there a way to keep katas private?

  • Default User Avatar

    Ok, thank you, I feel so stupid right now.
    I had just read title and example cases assuming the title was fully descriptive about that.

  • Default User Avatar

    to_camel_case('The-Stealth-Warrior') did not return correct value: 'theStealthWarrior' should equal 'TheStealthWarrior'

    Unless I am missing something, i think that 'TheStealthWarrior' isnt camelCase, but PascalCase.
    Is it just me or this test case is wrong?