The Stash documentation on pull requests does not detail in any way what exactly happens when you 'decline' a pull request. If I am added as a reviewer and I have a comment on the pull request, should I decline the pull request as well? Or just add a comment and wait until the pull request is updated again for my comment?
Declining a pull request is an indication that the pull request should probably never be merged to the destination branch. You might decline a pull request if: there are serious architectural problems with the incoming changes, there are philosophical conflicts in the changes or the code changes are no longer relevant (or will be incorporated in a few months time, perhaps).
If the pull request requires rework, then we suggest leaving the pull request open until those changes are pushed to the source branch, when the pull request will be updated automatically and show the new changes. You will be notified when changes are made to the pull request.
Hope that helps,
The problem with this workflow is there's no effective way to say "I'm done reviewing and this is not approved". Users will be notified of each comment you make and have no way of knowing whether or not you've completed your review or were pulled into a meeting halfway through it.
As a project manager, I have discovered that different developers want to bring their previous branching method with them when they join the team. Some developers are used to performing individual wo...
Connect with like-minded Atlassian users at free events near you!Find a group
Connect with like-minded Atlassian users at free events near you!
Unfortunately there are no AUG chapters near you at the moment.Start an AUG
You're one step closer to meeting fellow Atlassian users at your local meet up. Learn more about AUGs