-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Deduplicate - build a chessboard #82
Comments
I think the 2. is the most interesting due to a slightly added complexity, and unicode characters. It's also the most complete. The input validation requirement sucks, but maybe can be removed, or left as is. The unicode characters are reported to create some problems in one of languages. My vote is to keep 2. as the most interesting and most complete, and fix if needed. 1., 3., and 4. are simpler than 2., but still very similar. I think we don't need them if we keep 2. I think 5. is not a duplicate and should be kept. |
in spite of description for me ■ and □ works ok in Ruby in kata 2. in Mozilla |
I agree with keeping 2. The input validation is annoying, but it is the best option there. First one has a very strange output, 3 and 4 also have strange outputs and also single languages, 5 is needlessly python specific in its description and title. |
I didn't realize 5 was a bug-fixing kata tho at that time of posting(I even translated it in the wrong way😅) and considered the fact that it expects different output type, it is not a duplicate IMO.
Also, regarding 5. I'll fix my translation sooner or later. |
agree with keeping 2 and 5 |
Keep 2 (though the input validation and the output format can require adaptations for some languages, it can be done individually) for being the most interesting, remove 1,3,4. No opinion about 5. |
checkerboard / chessboard
"[r]"
or"[b]"
.■
and□
). Additional bit of difficulty is added because the color of the top left corner depends on input valuen
. Tests for invalid inputs like strings and negative numbers.'#'
.'*'
or'.'
. Boards can be rectangular.Conclusion:
The text was updated successfully, but these errors were encountered: