@yantonov
Hmm, weird, but that's fine. I've added some improvements to the tests since then, including random tests. :) Thank you!
P.S. I think Codewars allows me to approve the translation on my end. Let me know if you approve of the changes and that everything's ok - I can approve on my end.
What's new:
I. Fixed the bug in the tests - the function was originally erroneously expected to return true, given a 0 argument.
II. Added some tests for negative values as arguments.
III. Renamed the function to represent the conventional style of function-naming in Clojure.
@yantonov
Thank you!
The old solution was okay, - I only added in the new one in order to check for integers and accommodate for negative arguments.
I'm having the same issue with approving due to a 401 - I will ask for help on Gitter.
Everything is ok, approve if you can.
And thank you for changes to this old inelegant solution!
@yantonov
Hmm, weird, but that's fine. I've added some improvements to the tests since then, including random tests. :) Thank you!
P.S. I think Codewars allows me to approve the translation on my end. Let me know if you approve of the changes and that everything's ok - I can approve on my end.
ok, but cant approve these changes because of 401 for post {"success":false,"reason":"unauthorized"}
What's new:
I. Fixed the bug in the tests - the function was originally erroneously expected to return
true
, given a0
argument.II. Added some tests for negative values as arguments.
III. Renamed the function to represent the conventional style of function-naming in Clojure.
Missing clojure examples within description
Missing Clojure examples within description