Autolabel new Pull Requests

Author: javiereguiluz

src/AppBundle/Controller/WebhookController.php

@@ -40,7 +40,9 @@ public function githubAction(Request $request)
                         $responseData = [
                             'pull_request' => $data['pull_request']['number'],
                             'status_change' => $listener->handlePullRequestCreatedEvent(
-                                $data['pull_request']['number']
+                                $data['pull_request']['number'],
+                                $data['pull_request']['title'],
+                                $data['pull_request']['body']
                             ),
                         ];
                         break;

src/AppBundle/Issues/GitHub/GitHubStatusApi.php

@@ -78,6 +78,20 @@ public function setIssueStatus($issueNumber, $newStatus)
         }
     }
 
+    /**
+     * Replaces the existing issue labels (if any) with the given array of
+     * new labels. Use an empty array to remove all the existing labels.
+     *
+     * @param int    $issueNumber The GitHub issue number
+     * @param array  $newLabels
+     */
+    public function setIssueLabels($issueNumber, array $newLabels)
+    {
+        foreach ($newLabels as $label) {
+            $this->labelsApi->addIssueLabel($issueNumber, $label);
+        }
+    }
+
     public function getIssueStatus($issueNumber)
     {
         $currentLabels = $this->labelsApi->getIssueLabels($issueNumber);

src/AppBundle/Issues/IssueListener.php

@@ -58,14 +58,39 @@ public function handleCommentAddedEvent($issueNumber, $comment)
      * Adds a "Needs Review" label to new PRs.
      *
      * @param int $prNumber The number of the PR
+     * @param int $prTitle  The title of the PR
+     * @param int $prBody   The full text description of the PR
      *
      * @return string The new status
      */
-    public function handlePullRequestCreatedEvent($prNumber)
+    public function handlePullRequestCreatedEvent($prNumber, $prTitle, $prBody)
     {
+        $prLabels = array();
+
+        // new PRs always require review
         $newStatus = Status::NEEDS_REVIEW;
+        $prLabels[] = $newStatus;
+
+        // the PR title usually indicates the affected component
+        if (preg_match('/^\[(?P<component>.*)\] .*/$', $prTitle, $matches)) {
+            $prLabels[] = $matches['component'];
+        }
+
+        // the PR body usually indicates if this is a Bug, Feature, BC Break or Deprecation
+        if (preg_match('/^\|\s*Bug fix?\s*\|\s*yes\s*$/', $prBody, $matches)) {
+            $prLabels[] = 'Bug';
+        }
+        if (preg_match('/^\|\s*New feature?\s*\|\s*yes\s*$/', $prBody, $matches)) {
+            $prLabels[] = 'Feature';
+        }
+        if (preg_match('/^\|\s*BC breaks?\s*\|\s*yes\s*$/', $prBody, $matches)) {
+            $prLabels[] = 'BC Break';
+        }
+        if (preg_match('/^\|\s*Deprecations?\s*\|\s*yes\s*$/', $prBody, $matches)) {
+            $prLabels[] = 'Deprecation';
+        }
 
-        $this->statusApi->setIssueStatus($prNumber, $newStatus);
+        $this->statusApi->setIssueLabels($prNumber, $prLabels);
 
         return $newStatus;
     }

src/AppBundle/Issues/NullStatusApi.php

@@ -15,6 +15,10 @@ public function setIssueStatus($issueNumber, $newStatus)
     {
     }
 
+    public function setIssueLabels($issueNumber, array $newLabels)
+    {
+    }
+
     public function getNeedsReviewUrl()
     {
     }

src/AppBundle/Issues/StatusApi.php

@@ -13,5 +13,7 @@ public function getIssueStatus($issueNumber);
 
     public function setIssueStatus($issueNumber, $newStatus);
 
+    public function setIssueLabels($issueNumber, array $newLabels);
+
     public function getNeedsReviewUrl();
 }