Skip to content

Commit c75b520

Browse files
committed
phpcs
1 parent 79d5294 commit c75b520

File tree

5 files changed

+10
-18
lines changed

5 files changed

+10
-18
lines changed

src/AppBundle/Issues/NullStatusApi.php

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,5 @@ public function getIssueStatus($issueNumber, Repository $repository)
1515

1616
public function setIssueStatus($issueNumber, $newStatus, Repository $repository)
1717
{
18-
1918
}
2019
}

src/AppBundle/Subscriber/AutoLabelPRFromContentSubscriber.php

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@
88
use Symfony\Component\EventDispatcher\EventSubscriberInterface;
99

1010
/**
11-
* Looks at new pull requests and auto-labels based on text
11+
* Looks at new pull requests and auto-labels based on text.
1212
*/
1313
class AutoLabelPRFromContentSubscriber implements EventSubscriberInterface
1414
{
@@ -61,7 +61,7 @@ public function onPullRequest(GitHubEvent $event)
6161

6262
$event->setResponseData(array(
6363
'pull_request' => $prNumber,
64-
'pr_labels' => $prLabels
64+
'pr_labels' => $prLabels,
6565
));
6666
}
6767

@@ -82,7 +82,7 @@ private function extractLabels($prTitle)
8282
}
8383

8484
/**
85-
* TODO: get valid labels from the repository via GitHub API
85+
* TODO: get valid labels from the repository via GitHub API.
8686
*/
8787
private function getValidLabels()
8888
{
@@ -107,4 +107,4 @@ public static function getSubscribedEvents()
107107
GitHubEvents::PULL_REQUEST => 'onPullRequest',
108108
);
109109
}
110-
}
110+
}

src/AppBundle/Tests/Subscriber/AutoLabelPRFromContentSubscriberTest.php

Lines changed: 6 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -4,15 +4,11 @@
44

55
use AppBundle\Event\GitHubEvent;
66
use AppBundle\GitHubEvents;
7-
use AppBundle\Issues\Status;
87
use AppBundle\Repository\Repository;
98
use AppBundle\Subscriber\AutoLabelPRFromContentSubscriber;
10-
use AppBundle\Subscriber\BugLabelNewIssueSubscriber;
11-
use AppBundle\Subscriber\NeedsReviewNewPRSubscriber;
12-
use AppBundle\Subscriber\StatusChangeByCommentSubscriber;
139
use Symfony\Component\EventDispatcher\EventDispatcher;
1410

15-
class AutLabelPRFromContentSubscriberTest extends \PHPUnit_Framework_TestCase
11+
class AutoLabelPRFromContentSubscriberTest extends \PHPUnit_Framework_TestCase
1612
{
1713
private $autoLabelSubscriber;
1814

@@ -56,7 +52,7 @@ public function testAutoLabel($prTitle, $prBody, array $expectedNewLabels)
5652
'pull_request' => array(
5753
'number' => 1234,
5854
'title' => $prTitle,
59-
'body' => $prBody
55+
'body' => $prBody,
6056
),
6157
), $this->repository);
6258

@@ -80,7 +76,7 @@ public function getPRTests()
8076
FOO
8177
EOF
8278
,
83-
[]
79+
[],
8480
];
8581

8682
$tests[] = [
@@ -101,7 +97,7 @@ public function getPRTests()
10197
| Doc PR | n/a
10298
EOF
10399
,
104-
['Bug']
100+
['Bug'],
105101
];
106102

107103
$tests[] = [
@@ -122,15 +118,15 @@ public function getPRTests()
122118
| Doc PR | n/a
123119
EOF
124120
,
125-
['Bug', 'Feature', 'BC Break', 'Deprecation']
121+
['Bug', 'Feature', 'BC Break', 'Deprecation'],
126122
];
127123

128124
$tests[] = [
129125
'[Asset][BC Break][Fake] Extracting from title [Bug]',
130126
<<<EOF
131127
EOF
132128
,
133-
['Asset', 'BC Break', 'Bug']
129+
['Asset', 'BC Break', 'Bug'],
134130
];
135131

136132
return $tests;

src/AppBundle/Tests/Subscriber/BugLabelNewIssueSubscriberTest.php

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -7,8 +7,6 @@
77
use AppBundle\Issues\Status;
88
use AppBundle\Repository\Repository;
99
use AppBundle\Subscriber\BugLabelNewIssueSubscriber;
10-
use AppBundle\Subscriber\NeedsReviewNewPRSubscriber;
11-
use AppBundle\Subscriber\StatusChangeByCommentSubscriber;
1210
use Symfony\Component\EventDispatcher\EventDispatcher;
1311

1412
class BugLabelNewIssueSubscriberTest extends \PHPUnit_Framework_TestCase

src/AppBundle/Tests/Subscriber/NeedsReviewNewPRSubscriberTest.php

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,6 @@
77
use AppBundle\Issues\Status;
88
use AppBundle\Repository\Repository;
99
use AppBundle\Subscriber\NeedsReviewNewPRSubscriber;
10-
use AppBundle\Subscriber\StatusChangeByCommentSubscriber;
1110
use Symfony\Component\EventDispatcher\EventDispatcher;
1211

1312
class NeedsReviewNewPRSubscriberTest extends \PHPUnit_Framework_TestCase

0 commit comments

Comments
 (0)