Create
cancel
Showing results for 
Search instead for 
Did you mean: 
Sign up Log in

Do Pull Requests update when you push to a remote branch that has an open pull request?

Kevin Schumacher January 9, 2014

We are using a feature-branch based workflow. This is how we (want to) operate:

1. Create a branch locally on my dev machine.

2. Make one or more commits to my local branch.

3. Push branch to remote (Stash)

4. Create pull request. Initiate code review. Add comments to pull request.

5. Developer(s) make additional changes to the branch, and push those changes to the remote.

6. See the updated pull request, code review one more time, and then merge in if all is well (otherwise, go back to step 5).

For us, we get stuck at step 6. Our devs are able to push changes to the branch that's got a PR out, but the PR doesn't get updated. We try looking at the PR but it never updates. If I hit merge, it merges from the code at step 4 (before our review-initiated changes).

Is this a bug or the way it's supposed to work? We want to use Pull Requests the way GitHub uses pull requests (if you've read that article) -- lots of discussion around the work (including changes to the work) present in the pull request.

We were on 2.8.3 but have just upgraded to 2.10.1 and are still noticing this issue.

2 answers

1 accepted

0 votes
Answer accepted
cofarrell
Rising Star
Rising Star
Rising Stars are recognized for providing high-quality answers to other users. Rising Stars receive a certificate of achievement and are on the path to becoming Community Leaders.
January 9, 2014

Hi Kevin,

That is definitely not intended functionality, you should be able to use Stash like you would Github, but it's not a bug either. It's likely to be this known issue:

https://confluence.atlassian.com/display/STASHKB/Pull+Requests+Not+Reflecting+Changes+Pushed+to+Remote+Branch+After+an+Upgrade

The required Git hooks on your Stash server have obviously been corrupted in some way. Please talk to your sysadmin and follow the above article. If you get stuck please raise a support ticket so we can help you fix the problem.

Cheers,

Charles

0 votes
Kevin Schumacher January 9, 2014

Charles --

It was related to that issue. For some reason my hook scripts were not executable. Thank you for pointing me to that KB article!

Suggest an answer

Log in or Sign up to answer
TAGS
AUG Leaders

Atlassian Community Events