Ad
  • Custom User Avatar

    Thank you for the approval and your investigation. I like your kata a lot, keep up with the good work! For me, this was also the very first translation and pretty much enjoyed it. :)

  • Custom User Avatar

    Perfect.

    I tried to approve the translation, but i get
    "Description cannot be approved, recent changes from related record must be merged first."

    I have to admit i indeed made a slight change to the description (adding code-block markers around the
    examples), didn't know that could lead to trouble (my first kata, so i'm new to the whole process). So i do not know exactly what we need to do now, i tried rolling back my changes but that didn't help. I know that you have a APPROVE button on your translation, maybe that one is used to merge in my changes first ? Otherwise we would need to find a way to get my changes into your description (a lot of fuzz about 12 little accents :( )

    EDIT: I've found the info, on the translation page they say it needs a fork in a case like this. So i forked, put in the changes in the description and now your translation is online.

  • Custom User Avatar
  • Custom User Avatar

    Hi ignotus87,
    nice translation !

    Could you have a look at Voile's comment on the problem of testcases hitting the rounding border and make the test dropping these cases ? Otherwise people might experience correct solutions to be rejected due to floating point errors.