This is a kumite, not a fork; it shows things but you can't change them from here. If you like the changes ( you can see the exact ones with the Diff tab ), you'll have to fork or edit yourself ( or I could do it, but I'd also have so edit or fork ).

Updated the description and made examples language agnostic:
https://www.codewars.com/kumite/64bdaea4aaff1a94a8182b1d?sel=64c2ee3f5743822369f48368

Fixed.

Looks like I misread the JS version. Tests are fixed now.

Lua's require function works by implicitly treating the target file as the body of a function, the exported values are the values returned by calling that function. If the file is only returning a single function, this is the typical way of defining it. You could assign the function to a local and return the local if you really wanted to, but that's just an unnecessary layer of indirection if that's the only time the function is referenced by that name.

Not sure if I updated it correctly, but here is a fork with updated description: https://www.codewars.com/kumite/64bd5b298e1e959bea90ee2a?sel=64bda0ecaaff1a92ca182aa7

https://github.com/codewars/content-issues/wiki/List-of-PHP-Kata-to-Update

Note how its mentioned only in some snippets, not all of them. Don't ask me why its needed, im not a PHP person.

Possibly the info would have to be added in a better place in docs

Some people in contrary ask to add <?php in the beginning of solution and example solution to approve the translation. Unfortunately, I can't find recommendations regarding PHP in Codewars docs.

"filled - empty" is definetely better =)

First, I didn't read the whole description. Second, I had to use lowercase "p" for ";".