Kata

Ruby: only 3 fixed tests, and no random tests

Python translation with random tests

I would say its 8kyu not 7kyu really. If it makes any difference :P

The problem should work with or without the exclamation mark at the end, because it's just frustrating that an error pops just because I'm missing an exclamation mark.

In C#, there are only seven fixed tests. Needs random tests.

In C#, the expected and actual fields for tests are backwards.

Additionally, the method name greet should be Greet, as public method names are PascalCased in C#.

This kata isn't working properly in ruby toward the greet("") (if name is empty ). Says it expect nil and...When it gets nil it doesn't work.

Can't use ES6 arrow function as greet has already been defined

• This kata should should change the input into a string instead of forcing you to hard code it & throw an error for non string values.
• "hello" should not be case-sensitive
• Additional Test cases are needed to test non-string values It is nice to see some more beginner kata's since a lot of the 8kyu kata's are very hard for beginners

The comment in test case is too short. It should tell the me what is the expect output and my output so that I can understand what is wrong with my solution

Please add more test cases (before submitting).

cool!

The one tag (hello) is not useful in understanding the actual purpose of the kata.

Better failing test feedback would be an improvement, or more tests

Feel like this one is a little too simple and there are already challenges that touch on the same topic

Hello starting with capital H gives error, the kata only accepts hello with small H.

Says to return nil but expects null

null, not nil

var greet = function(name) { return "hello " + name + "!"; }; greet("Niks");

Passes until you submit?

I agree you should say return null, not nil, just to make it a little clearer

Should say to return null, not nil for non-"real names".

What do you mean about "real names"?

Needs more test cases. Description needs to use word 'null' instead of 'nil' for javascript.

Test case failures could be explained in more detail.

Tests give no information about the input or expected output.

Tests are insufficient, kata can be passed by hardcoding return string.

Kata is a duplicate of the many other 'say hello' kata.

Returning different data types from a function is bad practice.

You may want to be more explicit with your test cases. This is clearly for new people - provide some context for what might be going wrong if it doesn't pass. (Is it a string? Is it empty?)

This kata doesn't clearly describe the inputs/outputs it wants

what's expected if name 's empty ?