7 kyu
String Merge!
2,687 of 6,704cannse
Loading description...
Strings
Fundamentals
.avif)
This comment has been reported as {{ abuseKindText }}.
Show
This comment has been hidden. You can view it now .
This comment can not be viewed.
- |
- Reply
- Edit
- View Solution
- Expand 1 Reply Expand {{ comments?.length }} replies
- Collapse
- Spoiler
- Remove
- Remove comment & replies
- Report
{{ fetchSolutionsError }}
-
-
Your rendered github-flavored markdown will appear here.
-
Label this discussion...
-
No Label
Keep the comment unlabeled if none of the below applies.
-
Issue
Use the issue label when reporting problems with the kata.
Be sure to explain the problem clearly and include the steps to reproduce. -
Suggestion
Use the suggestion label if you have feedback on how this kata can be improved.
-
Question
Use the question label if you have questions and/or need help solving the kata.
Don't forget to mention the language you're using, and mark as having spoiler if you include your solution.
-
No Label
- Cancel
Commenting is not allowed on this discussion
You cannot view this solution
There is no solution to show
Please sign in or sign up to leave a comment.
python new test framework
Approved
COBOL translation + description made language agnostic + description slightly reworded ("word" -> "string", "letter" -> "character", since non alphabetic strings and characters are included; author is inactive).
C translation (author gone)
Approved :)
updated ruby translation kumited -- please check and approve (note: author is inactive)
approved
JS: Node v12 should be used along with the appropriate assertion tools (Mocha + Chai)
done there
C# translation added, please check and approve.
PHP:
Whoop, sorry. Updated and updated description for PHP code block
PHP translation, please check and approve :-)
Approved.
Could you approve my ruby translation?
Seems you didn't see my comment: description merge conflicts (@anter69)
@anter: when you'll fork your translation, could you also add the PHP code blocks to the description?
some languages are lacking random tests (java, at least)
added to Java, the other languages had them
lettershould becharin Java. Can someone approve the fork? https://www.codewars.com/kumite/5a704efcb17101de180000ce?sel=5be2cc36b3abcc00f90001e4done
There's still a problem, sample tests pass
letteras string, not char.forked
Ruby translation kumited -- pelase check and approve.
Merge conflict, sorry
see above
Updated python function name and description.
You should probably also specify if the merging should be case sensitive or not.
I'm assuming the assumption is that the two strings will always share the letter?
Great point! I've updated the description to include this assumption.
This comment has been hidden.
This comment has been hidden.
I would add to that the other type of edge case: what to do when the
lettershows up several times in a string?Description said:
; ) )
;-o (I'm not enough careful when I read descriptions of easy katas, it seems...)
This comment has been hidden.
all fixed by now, closing