Apply fixes from StyleCI

app/autoload.php

@@ -4,7 +4,7 @@
 use Composer\Autoload\ClassLoader;
 
 /**
- * @var ClassLoader $loader
+ * @var ClassLoader
  */
 $loader = require __DIR__.'/../vendor/autoload.php';
 

src/Regis/Application/Command/Github/Inspection/InspectPullRequest.php

@@ -27,4 +27,4 @@ public function getPullRequest(): PullRequest
     {
         return $this->pullRequest;
     }
-}
+}

src/Regis/Application/Command/Github/Inspection/SavePullRequestReport.php

@@ -26,4 +26,4 @@ public function getReport(): Entity\Inspection\Report
     {
         return $this->report;
     }
-}
+}

src/Regis/Application/Command/Github/Inspection/SchedulePullRequest.php

@@ -19,4 +19,4 @@ public function getPullRequest(): PullRequest
     {
         return $this->pullRequest;
     }
-}
+}

src/Regis/Application/Command/Github/Inspection/SendViolationsAsComments.php

@@ -27,4 +27,4 @@ public function getPullRequest(): Model\Github\PullRequest
     {
         return $this->pullRequest;
     }
-}
+}

src/Regis/Application/Command/Github/Repository/Create.php

@@ -33,4 +33,4 @@ public function getSharedSecret()
     {
         return $this->sharedSecret;
     }
-}
+}

src/Regis/Application/Command/Github/Repository/UpdateConfiguration.php

@@ -26,4 +26,4 @@ public function getNewSharedSecret(): string
     {
         return $this->newSharedSecret;
     }
-}
+}

src/Regis/Application/Command/Github/Webhook/Create.php

@@ -36,4 +36,4 @@ public function getCallbackUrl(): string
     {
         return $this->callbackUrl;
     }
-}
+}

src/Regis/Application/Command/SecureCommand.php

@@ -1,12 +1,12 @@
 <?php
 
-declare(strict_types = 1);
+declare(strict_types=1);
 
 namespace Regis\Application\Command;
 
 use Regis\Domain\Entity;
 
 interface SecureCommand
 {
-    function executionAuthorizedFor(Entity\User $user): bool;
-}
+    public function executionAuthorizedFor(Entity\User $user): bool;
+}

src/Regis/Application/Command/SecureCommandBySpecification.php

@@ -1,6 +1,6 @@
 <?php
 
-declare(strict_types = 1);
+declare(strict_types=1);
 
 namespace Regis\Application\Command;
 
@@ -10,7 +10,7 @@
 
 interface SecureCommandBySpecification
 {
-    static function executionAuthorizedFor(Entity\User $user): Specification;
+    public static function executionAuthorizedFor(Entity\User $user): Specification;
 
-    function getTargetToSecure();
-}
+    public function getTargetToSecure();
+}

src/Regis/Application/Command/Team/AddMember.php

@@ -40,4 +40,4 @@ public function getTargetToSecure()
     {
         return $this->team;
     }
-}
+}

src/Regis/Application/Command/Team/AddRepository.php

@@ -40,4 +40,4 @@ public function getTargetToSecure()
     {
         return $this->team;
     }
-}
+}

src/Regis/Application/Command/Team/Create.php

@@ -26,4 +26,4 @@ public function getName(): string
     {
         return $this->name;
     }
-}
+}

src/Regis/Application/Command/Team/Leave.php

@@ -40,4 +40,4 @@ public function getTargetToSecure()
     {
         return $this->team;
     }
-}
+}

src/Regis/Application/Command/Team/RemoveMember.php

@@ -40,4 +40,4 @@ public function getTargetToSecure()
     {
         return $this->team;
     }
-}
+}

src/Regis/Application/Command/Team/RemoveRepository.php

@@ -40,4 +40,4 @@ public function getTargetToSecure()
     {
         return $this->team;
     }
-}
+}

src/Regis/Application/Command/User/CreateAdmin.php

@@ -31,4 +31,4 @@ public function getEmail(): string
     {
         return $this->email;
     }
-}
+}

src/Regis/Application/Command/User/CreateOrUpdateUser.php

@@ -38,4 +38,4 @@ public function getAccessToken(): string
     {
         return $this->accessToken;
     }
-}
+}

src/Regis/Application/CommandHandler/Github/Inspection/InspectPullRequest.php

@@ -60,4 +60,4 @@ private function dispatch(string $eventName, Event $event)
     {
         $this->dispatcher->dispatch($eventName, new Event\DomainEventWrapper($event));
     }
-}
+}

src/Regis/Application/CommandHandler/Github/Inspection/SavePullRequestReport.php

@@ -24,4 +24,4 @@ public function handle(Command\Github\Inspection\SavePullRequestReport $command)
 
         $this->inspectionsRepository->save($inspection);
     }
-}
+}

src/Regis/Application/CommandHandler/Middleware/Security.php

@@ -1,6 +1,6 @@
 <?php
 
-declare(strict_types = 1);
+declare(strict_types=1);
 
 namespace Regis\Application\CommandHandler\Middleware;
 
@@ -53,4 +53,4 @@ private function executionAuthorized($command): bool
 
         return $authorized;
     }
-}
+}

src/Regis/Application/CommandHandler/Team/AddMember.php

@@ -31,4 +31,4 @@ public function handle(Command\Team\AddMember $command)
             // The given user was already in the team, nothing to do!
         }
     }
-}
+}

src/Regis/Application/CommandHandler/Team/AddRepository.php

@@ -31,4 +31,4 @@ public function handle(Command\Team\AddRepository $command)
             // The given repo was already in the team, nothing to do!
         }
     }
-}
+}

src/Regis/Application/CommandHandler/Team/Create.php

@@ -23,4 +23,4 @@ public function handle(Command\Team\Create $command)
 
         $this->teamsRepo->save($team);
     }
-}
+}

src/Regis/Application/CommandHandler/Team/Leave.php

@@ -23,4 +23,4 @@ public function handle(Command\Team\Leave $command)
 
         $this->teamsRepo->save($team);
     }
-}
+}

src/Regis/Application/CommandHandler/Team/RemoveMember.php

@@ -27,4 +27,4 @@ public function handle(Command\Team\RemoveMember $command)
 
         $this->teamsRepo->save($team);
     }
-}
+}

src/Regis/Application/CommandHandler/Team/RemoveRepository.php

@@ -27,4 +27,4 @@ public function handle(Command\Team\RemoveRepository $command)
 
         $this->teamsRepo->save($team);
     }
-}
+}

src/Regis/Application/CommandHandler/User/CreateAdmin.php

@@ -33,4 +33,4 @@ public function handle(Command\User\CreateAdmin $command)
 
         $this->usersRepo->save($user);
     }
-}
+}

src/Regis/Application/CommandHandler/User/CreateOrUpdateUser.php

@@ -31,4 +31,4 @@ public function handle(Command\User\CreateOrUpdateUser $command)
 
         return $user;
     }
-}
+}

src/Regis/Application/Event.php

@@ -14,5 +14,5 @@ interface Event
     const INSPECTION_FINISHED = 'inspection_finished';
     const INSPECTION_FAILED = 'inspection_failed';
 
-    function getEventName(): string;
-}
+    public function getEventName(): string;
+}

src/Regis/Application/Event/DomainEventWrapper.php

@@ -20,4 +20,4 @@ public function getDomainEvent(): DomainEvent
     {
         return $this->domainEvent;
     }
-}
+}

src/Regis/Application/Event/InspectionFailed.php

@@ -40,4 +40,4 @@ public function getEventName(): string
     {
         return Event::INSPECTION_FAILED;
     }
-}
+}

src/Regis/Application/Event/InspectionFinished.php

@@ -40,4 +40,4 @@ public function getEventName(): string
     {
         return Event::INSPECTION_FINISHED;
     }
-}
+}

src/Regis/Application/Event/InspectionStarted.php

@@ -33,4 +33,4 @@ public function getEventName(): string
     {
         return Event::INSPECTION_STARTED;
     }
-}
+}

src/Regis/Application/Event/PullRequestClosed.php

@@ -25,4 +25,4 @@ public function getEventName(): string
     {
         return Event::PULL_REQUEST_CLOSED;
     }
-}
+}

src/Regis/Application/Event/PullRequestOpened.php

@@ -25,4 +25,4 @@ public function getEventName(): string
     {
         return Event::PULL_REQUEST_OPENED;
     }
-}
+}

src/Regis/Application/Event/PullRequestSynced.php

@@ -39,4 +39,4 @@ public function getEventName(): string
     {
         return Event::PULL_REQUEST_SYNCED;
     }
-}
+}

src/Regis/Application/EventListener/PullRequestInspectionReportListener.php

@@ -36,4 +36,4 @@ public function onInspectionFinished(Event\DomainEventWrapper $event)
             $domainEvent->getReport()
         ));
     }
-}
+}

src/Regis/Application/EventListener/PullRequestListener.php

@@ -48,4 +48,4 @@ public function onPullRequestClosed(Event\DomainEventWrapper $event)
         // TODO should be in a command
         $this->violationsCache->clear($domainEvent->getPullRequest());
     }
-}
+}

src/Regis/Application/EventListener/PullRequestReviewCommentsListener.php

@@ -36,4 +36,4 @@ public function onInspectionFinished(Event\DomainEventWrapper $event)
             $domainEvent->getPullRequest()
         ));
     }
-}
+}

src/Regis/Application/Github/Client.php

@@ -16,9 +16,13 @@ interface Client
     const READONLY_KEY = 'readonly_key';
     const WRITE_KEY = 'write_key';
 
-    function setIntegrationStatus(Model\PullRequest $pullRequest, IntegrationStatus $status);
-    function addDeployKey(string $owner, string $repository, string $title, string $key, string $type);
-    function createWebhook(string $owner, string $repository, string $url, $secret = null);
-    function sendComment(Model\PullRequest $pullRequest, Model\ReviewComment $comment);
-    function listRepositories(): \Traversable;
-}
+    public function setIntegrationStatus(Model\PullRequest $pullRequest, IntegrationStatus $status);
+
+    public function addDeployKey(string $owner, string $repository, string $title, string $key, string $type);
+
+    public function createWebhook(string $owner, string $repository, string $url, $secret = null);
+
+    public function sendComment(Model\PullRequest $pullRequest, Model\ReviewComment $comment);
+
+    public function listRepositories(): \Traversable;
+}

src/Regis/Application/Github/ClientFactory.php

@@ -8,6 +8,7 @@
 
 interface ClientFactory
 {
-    function createForRepository(Entity\Github\Repository $repository): Client;
-    function createForUser(Entity\User $user): Client;
+    public function createForRepository(Entity\Github\Repository $repository): Client;
+
+    public function createForUser(Entity\User $user): Client;
 }

src/Regis/Application/Github/EventTransformer.php

@@ -28,9 +28,9 @@ public function transform(Request $request)
 
             if ($payload['action'] === self::ACTION_OPEN_PULL_REQUEST) {
                 return $this->transformPullRequestOpened($payload);
-            } else if ($payload['action'] === self::ACTION_SYNC_PULL_REQUEST) {
+            } elseif ($payload['action'] === self::ACTION_SYNC_PULL_REQUEST) {
                 return $this->transformPullRequestSynced($payload);
-            } else if ($payload['action'] === self::ACTION_CLOSE_PULL_REQUEST) {
+            } elseif ($payload['action'] === self::ACTION_CLOSE_PULL_REQUEST) {
                 return $this->transformPullRequestClosed($payload);
             }
         }
@@ -72,4 +72,4 @@ private function transformPullRequest(array $payload): Model\Github\PullRequest
             new Model\Git\Revisions($payload['pull_request']['base']['sha'], $payload['pull_request']['head']['sha'])
         );
     }
-}
+}

src/Regis/Application/Github/Exception/EventNotHandled.php

@@ -6,4 +6,4 @@
 
 class EventNotHandled extends \RuntimeException
 {
-}
+}

src/Regis/Application/Github/IntegrationStatus.php

@@ -1,6 +1,6 @@
 <?php
 
-declare(strict_types = 1);
+declare(strict_types=1);
 
 namespace Regis\Application\Github;
 
@@ -36,4 +36,4 @@ public function getTargetUrl()
     {
         return $this->targetUrl;
     }
-}
+}

src/Regis/Application/Inspection.php

@@ -8,7 +8,7 @@
 
 interface Inspection
 {
-    function getType(): string;
+    public function getType(): string;
 
-    function inspectDiff(Model\Git\Diff $diff): \Traversable;
-}
+    public function inspectDiff(Model\Git\Diff $diff): \Traversable;
+}