Is_valid_ip isn't a proper Go naming convention. Consider chaning it to IsValidIp

To Go version is still broken.

codewarrior/kata

./setup.go:3: imported and not used: "strings"
./setup.go:7: missing return at end of function
./solution.go:21: ToNato redeclared in this block
previous declaration at ./setup.go:5

Sorry about that, I was typing in Python before that, you know how it is switching between languages. Secondly, I tried to make the change but i'm recieving this error -> "Description cannot be approved, recent changes from related record must be merged first."

That would be great, I liked this problem.

Left a Java translation if you want to take a look at it.

I have the answer done in Java, but i'm having trouble with setting up the test cases in Java. The setup is significantly different then Python. So If you want to modify what I submitted and have a Java solution here's what I got so far.

I have the answer done in Java, but i'm having trouble with setting up the test cases in Java. The setup is significantly different then Python. So If you want to modify what I submitted and have a Java solution, this is as far as I got.

I'm sorry I see what I did wrong. I placed the class and list instances in the in the preloaded section so it should work again. However, a linkedlist instance will always be passed. Node instaces are created within the linkeslist instance. That's how I implemented it. I like that implementation. I will hide the class and and created linked list instances in the preloaded section. That will make things cleaner. You can access the nodes still via next, head, and data.

Is this what you meant. I removed it from preloaded coded and used it to build my test cases and example cases.

I'll see what I can do :-)

I mentioned it in the description along with how to access the list's head if it wasn't clear already.

Okay, made the change cur_node is now head. I hide the LinkedList and Node Structure by placing it in the preloaded code. I change the search_k_from_end method to take a linkedlist instance and k. I informed the user how to get the head of the linked list which is all that is need to solve the problem.

Nice i'll get on those suggestions sometime today. Thanks guys.

You use None all the time in defined method aruguments, in basic struts, python classes. It's common enough to have a check for it.