I think `get_commit_status_warning` is a better name for this since it returns a string. Also, is defaulting `status = ''` necessary? It doesn't make any sense to call this function without this argument.