Is this actually an `if Children is not None` check? I think it's better to be explicit that it's not testing whether `Children` is empty (as that would already be handled by the loop).