Please do not stringify the pyramids prior to assertions; instead, assert on the arrays themselves and use the optional JUnit String message parameter if you want to provide pretty-printing
It is possible for the kata description to be updated on the kata itself, after you have updated the description here. When this happens, if there are merge conflicts an additional fork will need to be made to merge the differences together.
Basically, (if I understand this correctly), if a description gets changed after a translation is created, it will confilict upon attempt of approval. So, a translator should (at best, when possible) always update the description version for the translation prior to approval.
actually i don't know about when and why merge conflicts occur, so i do not really know
where to look for what kind of error and what i need to do to reproduce the merge conflict.
google gave me https://github.com/Codewars/codewars.com/issues/1949
so i forked the translation
approved some while ago
.
tests changed,thanks
Corrected, thanks!
A few minor typos in the description:
python new test framework is required. updated in this fork
approved
Thanks for the suggestion, I changed the code of the sample tests and tests accordingly
Please do not stringify the pyramids prior to assertions; instead, assert on the arrays themselves and use the optional JUnit
String message
parameter if you want to provide pretty-printingThanks, that was helpful.
Help section says
so the fork should do the trick
Basically, (if I understand this correctly), if a description gets changed after a translation is created, it will confilict upon attempt of approval. So, a translator should (at best, when possible) always update the description version for the translation prior to approval.
actually i don't know about when and why merge conflicts occur, so i do not really know
where to look for what kind of error and what i need to do to reproduce the merge conflict.
google gave me
https://github.com/Codewars/codewars.com/issues/1949
so i forked the translation
merge conflict
This is an issue
Loading more items...