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

Post-hooks Unsuccessfully Triggered After Clicking Merge Button in a Pull Request

Ted Sheibar January 28, 2013

If a developer makes a direct push to a branch using a repository's Stash URL, the post-receive hook gets successfully triggered.

However, the same hook doesn't trigger if the developer uses the Merge button after his/her changes are code reviewed.

I thought that since the pull request merge happens server-side, the post-commit hook would be the correct place to have a repository trigger. However, having the repo trigger in both the post-receive and post-commit still didn't work after using the Merge button in a pull request.

Is this a bug? What hooks are called before and after the server-side merge in Stash?

4 answers

1 accepted

4 votes
Answer accepted
Josh Baird January 6, 2014

Is it still not possible to have a hook that is triggered after a pull request merge?

1 vote
TimP
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 29, 2013

Hi rl_ted,

The mechanism that performs the pull request merge is actually based on a git fetch into the repository, which doesn't trigger receive hooks. Unfortunately there are currently no hooks that will be invoked automatically after a merge.

That said, if you want to trigger functionality based on a pull request merge, you can build a Stash plugin that consumes the PullRequestMergedEvent. There is also a java API for implementing commit-hooks in a plugin. If you're unfamiliar with Atlassian plugins, the Atlassian SDK is a good place to start.

I'd also keep an eye on STASH-2521, which tracks improved commit hooks support in Stash and should hit arelease in the near future.

cheers,

Tim

Ted Sheibar February 2, 2013

Thank you, Tim.

0 votes
TimP
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.
July 18, 2014

Hi guys,

Here's an update on my answer from last year:

Repository hooks implementing AsyncPostReceiveRepositoryHook will be called for both pushes and Pull Request merges. See https://developer.atlassian.com/stash/docs/latest/how-tos/repository-hooks.html for full details.

cheers,

Tim

0 votes
Daniel R
Atlassian Team
Atlassian Team members are employees working across the company in a wide variety of roles.
July 18, 2014

The improvement request that Tim originally provided has been split-up. Follow this improvement request instead: - https://jira.atlassian.com/browse/STASH-3247

Suggest an answer

Log in or Sign up to answer
TAGS
AUG Leaders

Atlassian Community Events