Uploaded image for project: 'Skara'
  1. Skara
  2. SKARA-2234

/reviewers N should remove ready status for merge pull requests

XMLWordPrintable

    • Icon: Bug Bug
    • Resolution: Fixed
    • Icon: P4 P4
    • 1.0
    • None
    • bots
    • None

      In this pr https://github.com/openjdk/jfx22u/pull/26, Kevin issued "/reviewers 1" in the merge pull request but skara didn't remove the ready label.
      Jfx22u was not configured with “reviewMerge”, so the reviewers jcheck were skipped. The reason we can see Change must be properly reviewed (1 review required, with at least 1 Reviewer) in the PR body is because of SKARA-1824.
      To resolve this issue, we should enable reviewMerge when a reviewers command is issued in a merge pull request.

            zsong Zhao Song
            zsong Zhao Song
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated:
              Resolved: