Uploaded image for project: 'Jenkins'
  1. Jenkins
  2. JENKINS-62924

pr-merge status notification sent to GitHub applies to multiple PRs if source branch has multiple PRs opened to different target branches

    XMLWordPrintable

    Details

    • Similar Issues:

      Description

      If multiple, simultaneous PRs are opened for single source branch to different target branches, e.g. a PR to dev and a PR to master, the pr-merge status notification received by GitHub will apply the status notification to all PRs when it should only apply to a single PR.

        Attachments

          Issue Links

            Activity

            Hide
            bitwiseman Liam Newman added a comment -

            Derrick Li
            I'm pretty sure this is because the plugin uses github commit status. JENKINS-55726 might also effect this.
            Valid bug, though not the most common scenario.

            Show
            bitwiseman Liam Newman added a comment - Derrick Li I'm pretty sure this is because the plugin uses github commit status. JENKINS-55726 might also effect this. Valid bug, though not the most common scenario.
            Hide
            drock Derrick Li added a comment -

            Understandable. A simple restriction can be implemented by failing the builds if multiple PRs are detected and informing the user on the PR (via GitHub's API).

            Show
            drock Derrick Li added a comment - Understandable. A simple restriction can be implemented by failing the builds if multiple PRs are detected and informing the user on the PR (via GitHub's API).
            Hide
            bitwiseman Liam Newman added a comment -

            Derrick Li
            Great! It sounds like you understand what is needed. Please submit a PR.

            Show
            bitwiseman Liam Newman added a comment - Derrick Li Great! It sounds like you understand what is needed. Please submit a PR.

              People

              Assignee:
              Unassigned Unassigned
              Reporter:
              drock Derrick Li
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Dates

                Created:
                Updated: