Skip to content
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

#428-add-support-to-pr:reviewer:approved-event #429

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

MayaPetter
Copy link

@MayaPetter MayaPetter commented Feb 15, 2021

adding support to reviewer approval event to update the build

Your checklist for this pull request

  • [x ] Make sure you are requesting to pull a topic/feature/bugfix branch (right side) and not your master branch!
  • [ x] Ensure that the pull request title represents the desired changelog entry
  • [ x] added SERVER_PULL_REQUEST_APPROVED event to NativeServerPullRequestHookProcessor.java and WebhookConfiguration.java
  • [x ] add-support-to-pr:reviewer:approved-event #428
  • Link to relevant pull requests, esp. upstream and downstream changes
  • [ x] Did you provide a test-case? That demonstrates feature works or fixes the issue.-
    tested locally on a multibranch job, triggered scanning on or approval on bitbucket server

@MayaPetter MayaPetter marked this pull request as draft February 15, 2021 17:56
@MayaPetter MayaPetter marked this pull request as ready for review February 15, 2021 21:11
Copy link
Author

@MayaPetter MayaPetter left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@wahammed
Copy link

wahammed commented May 4, 2021

Who can merge this?

@bitwiseman
Copy link
Contributor

bitwiseman commented Jul 8, 2021

@wahammed
There was a question on #428 as to whether this applies to Cloud. I think it does.
https://support.atlassian.com/bitbucket-cloud/docs/event-payloads/#Approved

Comment on lines -1 to -10
{
"values": [{
"date": "2018-09-21T14:53:12+00:00",
"message": "[CI] Release version 1.0.0",
"hash": "bf0e8b7962c024026ad01ae09d3a11732e26c0d4",
"author": {
"raw": "Builder <[email protected]>"
}
}]
}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why was this deleted? I assume this was deleted to make a test pass?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi, it was probably a mistake, I didn't try to delete it...

Copy link
Contributor

@bitwiseman bitwiseman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This event is also present in Bitbucket Cloud.
https://support.atlassian.com/bitbucket-cloud/docs/event-payloads/#Approved

And it was added to this codebase:
https://github.com/jenkinsci/bitbucket-branch-source-plugin/blob/master/src/main/java/com/cloudbees/jenkins/plugins/bitbucket/hooks/HookEventType.java#L62

But then it is never used. 😭

Please update this PR to add the Cloud version as well.

@jverhoeven
Copy link

@MayaPetter would you be so kind to complete this PR? We are running into the same issue and prefer to have this fixed in main plugin and not our own compiled one. Many thanks!

@jkurek1
Copy link

jkurek1 commented Jun 23, 2023

Hi,
Any progress on that feature? It would be really good to have it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants