search

jenkinsci / gogs-webhook-plugin

Jenkins Gogs Webhook
https://plugins.jenkins.io/gogs-webhook/
MIT License
79 stars 42 forks source link

Update GogsWebHook.java #20

Closed TheUltimateC0der closed 7 years ago

TheUltimateC0der commented 7 years ago

Added support for multibranch pipelines

pascalw commented 7 years ago

Thanks for the patch @S3NS4Ti0N I was looking for exactly this. Will you make the changes suggested by @fabienjuif? If you want I could also make the changes. Let me know.

pascalw commented 7 years ago

@S3NS4Ti0N @fabienjuif I pushed my changes here https://github.com/jenkinsci/gogs-webhook-plugin/compare/master...pascalw:feature/support-multibranch?expand=1 in case you're interested. I've refactored the Webhook class a bit and also made some changes to support branches with / chars in the branch name.

Let me know if you want me to create a pull request.

sanderv32 commented 7 years ago

Coming weekend I will test it and merge this PR

TheUltimateC0der commented 7 years ago

Whoop Whoop

sanderv32 commented 7 years ago

Merged the PR