It doesn't matter *that* much but I feel like this should be moved as late as possible, to ensure that you only update the revision if you're pretty sure the push is actually a success.