This looks like a fatuous infinite recursion; a function that calls itself with its own arguments !
Did you mean to delegate to some other function, e.g. s/NoCommit// ?