We're doing this in a couple of places. A helper named something like `get_branch_name_for_pr` may be more explicit. Also, is `pr` expected to be a number or the branch name? it's not clear from the docstring.