New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Call merge request web hook for each commits #8872

Merged
merged 1 commit into from May 13, 2015

Conversation

5 participants
@baloo
Contributor

baloo commented Feb 26, 2015

Execute merge request web hook when new commits comes in the source branch of the merge request.

@TeatroIO

This comment has been minimized.

Show comment
Hide comment
@TeatroIO

TeatroIO Feb 26, 2015

I've prepared a stage to preview changes. Open stage or view logs.

TeatroIO commented Feb 26, 2015

I've prepared a stage to preview changes. Open stage or view logs.

@baloo baloo changed the title from Execute merge request web hook upon commits to Call merge request web hook for each commits Feb 26, 2015

@jvanbaarsen

This comment has been minimized.

Show comment
Hide comment
@jvanbaarsen

jvanbaarsen May 9, 2015

Contributor

@randx What do you think about this?

Contributor

jvanbaarsen commented May 9, 2015

@randx What do you think about this?

@dzaporozhets

This comment has been minimized.

Show comment
Hide comment
@dzaporozhets

dzaporozhets May 11, 2015

Member

@jvanbaarsen I think it is useful for CI tools to get update when you push new code to MR source branch. I think we can accept this. But it needs CHANGELOG item and mention in docs

Member

dzaporozhets commented May 11, 2015

@jvanbaarsen I think it is useful for CI tools to get update when you push new code to MR source branch. I think we can accept this. But it needs CHANGELOG item and mention in docs

@jvanbaarsen

This comment has been minimized.

Show comment
Hide comment
@jvanbaarsen

jvanbaarsen May 11, 2015

Contributor

@baloo Can you please add Item to CHANGELOG and mention this in the docs? Ping me when ready :)

Contributor

jvanbaarsen commented May 11, 2015

@baloo Can you please add Item to CHANGELOG and mention this in the docs? Ping me when ready :)

@baloo

This comment has been minimized.

Show comment
Hide comment
@baloo

baloo May 11, 2015

Contributor

ping @jvanbaarsen

Tests looks broken in master, I rebased on master so CHANGELOG mention does not merge-conflict. I will rebase on new master once tests are fixed there so everything looks green for this PR.

Contributor

baloo commented May 11, 2015

ping @jvanbaarsen

Tests looks broken in master, I rebased on master so CHANGELOG mention does not merge-conflict. I will rebase on new master once tests are fixed there so everything looks green for this PR.

@jvanbaarsen

This comment has been minimized.

Show comment
Hide comment
@jvanbaarsen

jvanbaarsen May 11, 2015

Contributor

@baloo Thanks!

Contributor

jvanbaarsen commented May 11, 2015

@baloo Thanks!

Call merge request web hook for each commits
Call merge request web hook every times a new commit hits the
source branch

Signed-off-by: Arthur Gautier <baloo@gandi.net>
@baloo

This comment has been minimized.

Show comment
Hide comment
@baloo

baloo May 13, 2015

Contributor
Contributor

baloo commented May 13, 2015

jvanbaarsen added a commit that referenced this pull request May 13, 2015

Merge pull request #8872 from baloo/feature/execute_mr_webhooks_upon_…
…commit

Call merge request web hook for each commits

@jvanbaarsen jvanbaarsen merged commit 892158f into gitlabhq:master May 13, 2015

1 check passed

semaphoreci The build passed on Semaphore.
Details
@jvanbaarsen

This comment has been minimized.

Show comment
Hide comment
@jvanbaarsen

jvanbaarsen May 13, 2015

Contributor

Thanks

Contributor

jvanbaarsen commented May 13, 2015

Thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment