John1

Please merge to your master.
No need to have a return then a break. A return always exits the function so this break is never reached. Also it's a little bizarre to be returning a string or None. It might be more streamlined to raise a custom error return a status code (0, 1, -1, etc)