Fixed all style violations

Author: WyriHaximus

src/ApiSettings.php

@@ -18,7 +18,7 @@ final class ApiSettings
 {
     /**
      * Travis' Pusher App ID as found on: https://docs.travis-ci.com/api?http#external-apis
-     * Will automate the retrieval of that key later in the PR
+     * Will automate the retrieval of that key later in the PR.
      */
     const PUSHER_KEY = '5df8ac576dcccf4fd076';
 
@@ -50,11 +50,11 @@ final class ApiSettings
 
     /**
      * Get client options based on $token, $suffix, and $suppliedOptions.
-     * Will add auth middleware when $token isn't empty
+     * Will add auth middleware when $token isn't empty.
      *
-     * @param string $token
-     * @param string $suffix
-     * @param array $suppliedOptions
+     * @param  string $token
+     * @param  string $suffix
+     * @param  array  $suppliedOptions
      * @return array
      */
     public static function getOptions(

src/AsyncClient.php

@@ -22,11 +22,19 @@ final class AsyncClient implements AsyncClientInterface
     private $client;
 
     /**
-     * Create a new AsyncClient based on the loop and other options pass
+     * @param ClientInterface $client
+     */
+    private function __construct(ClientInterface $client)
+    {
+        $this->client = $client;
+    }
+
+    /**
+     * Create a new AsyncClient based on the loop and other options pass.
      *
-     * @param LoopInterface $loop
-     * @param string $token Instructions to fetch the token: https://blog.travis-ci.com/2013-01-28-token-token-token/
-     * @param array $options
+     * @param  LoopInterface $loop
+     * @param  string        $token   Instructions to fetch the token: https://blog.travis-ci.com/2013-01-28-token-token-token/
+     * @param  array         $options
      * @return AsyncClient
      */
     public static function create(
@@ -43,29 +51,22 @@ public static function create(
 
         $options = ApiSettings::getOptions($token, 'Async', $options);
         $client = Factory::create($loop, $options);
+
         return new self($client);
     }
 
     /**
      * Create an AsyncClient from a ApiClients\Foundation\ClientInterface.
      * Be sure to pass in a client with the options from ApiSettings and the Async namespace suffix.
      *
-     * @param ClientInterface $client
+     * @param  ClientInterface $client
      * @return AsyncClient
      */
     public static function createFromClient(ClientInterface $client): self
     {
         return new self($client);
     }
 
-    /**
-     * @param ClientInterface $client
-     */
-    private function __construct(ClientInterface $client)
-    {
-        $this->client = $client;
-    }
-
     /**
      * {@inheritdoc}
      */

src/AsyncClientInterface.php

@@ -2,12 +2,10 @@
 
 namespace ApiClients\Client\Travis;
 
-use ApiClients\Client\Travis\CommandBus\Command;
 use React\Promise\CancellablePromiseInterface;
 use React\Promise\PromiseInterface;
 use Rx\Observable;
 use Rx\ObservableInterface;
-use function ApiClients\Tools\Rx\unwrapObservableFromPromise;
 use function React\Promise\resolve;
 
 interface AsyncClientInterface
@@ -16,29 +14,29 @@ interface AsyncClientInterface
      * Fetch the given repository.
      * This will resolve a Resource\Async\Repository object.
      *
-     * @param string $repository
+     * @param  string                      $repository
      * @return CancellablePromiseInterface
      */
     public function repository(string $repository): CancellablePromiseInterface;
 
     /**
      * Fetch all the repositories linked to the authenticated user's account.
      * This will return a stream of Resource\Async\Repository objects.
-     * (Requires auth token to be passed to the client!)
+     * (Requires auth token to be passed to the client!).
      *
      * This function accepts an optional callable passed into a filter call
      * on the hooks observable before it fetches the repository information.
      * It could be used to only fetch new repositories.
      *
-     * @param callable|null $filter
+     * @param  callable|null       $filter
      * @return ObservableInterface
      */
     public function repositories(callable $filter = null): ObservableInterface;
 
     /**
      * Fetch information about the current authenticated user.
      * This will resolve a Resource\Async\User object.
-     * (Requires auth token to be passed to the client!)
+     * (Requires auth token to be passed to the client!).
      *
      * @return PromiseInterface
      */
@@ -48,9 +46,9 @@ public function user(): PromiseInterface;
      * Fetch the SSH key for the given repository ID.
      * This will resolve a Resource\Async\SSHKey object.
      * (Requires auth token to be passed to the client!)
-     * (Only available on Travis Pro: https://travis-ci.com/)
+     * (Only available on Travis Pro: https://travis-ci.com/).
      *
-     * @param int $id
+     * @param  int              $id
      * @return PromiseInterface
      */
     public function sshKey(int $id): PromiseInterface;
@@ -60,7 +58,7 @@ public function sshKey(int $id): PromiseInterface;
      * Hooks represent whether a repository is active or not,
      * their ID's match that of the repository they represent.
      * This will return a stream of Resource\Async\Hook objects.
-     * (Requires auth token to be passed to the client!)
+     * (Requires auth token to be passed to the client!).
      *
      * @return ObservableInterface
      */
@@ -69,7 +67,7 @@ public function hooks(): ObservableInterface;
     /**
      * Fetch a stream of which users and orgs the currently authenticated user has access to.
      * This will return a stream of Resource\Async\Account objects.
-     * (Requires auth token to be passed to the client!)
+     * (Requires auth token to be passed to the client!).
      *
      * @return ObservableInterface
      */
@@ -78,7 +76,7 @@ public function accounts(): ObservableInterface;
     /**
      * Fetch a list of currently active broadcast. Used by the Travis team to spread news on the site.
      * This will return a stream of Resource\Async\Broadcast objects.
-     * (Requires auth token to be passed to the client!)
+     * (Requires auth token to be passed to the client!).
      *
      * @return ObservableInterface
      */

src/Client.php

@@ -3,15 +3,14 @@
 
 namespace ApiClients\Client\Travis;
 
+use ApiClients\Client\Travis\Resource\RepositoryInterface;
+use ApiClients\Client\Travis\Resource\SSHKeyInterface;
+use ApiClients\Client\Travis\Resource\UserInterface;
 use ApiClients\Foundation\Factory;
 use React\EventLoop\Factory as LoopFactory;
 use React\EventLoop\LoopInterface;
 use Rx\React\Promise;
-use ApiClients\Client\Travis\Resource\RepositoryInterface;
-use ApiClients\Client\Travis\Resource\SSHKeyInterface;
-use ApiClients\Client\Travis\Resource\UserInterface;
 use function Clue\React\Block\await;
-use function React\Promise\resolve;
 
 final class Client implements ClientInterface
 {
@@ -26,10 +25,20 @@ final class Client implements ClientInterface
     private $asyncClient;
 
     /**
-     * Create a new AsyncClient based on the loop and other options pass
+     * @param LoopInterface        $loop
+     * @param AsyncClientInterface $asyncClient
+     */
+    private function __construct(LoopInterface $loop, AsyncClientInterface $asyncClient)
+    {
+        $this->loop = $loop;
+        $this->asyncClient = $asyncClient;
+    }
+
+    /**
+     * Create a new AsyncClient based on the loop and other options pass.
      *
-     * @param string $token Instructions to fetch the token: https://blog.travis-ci.com/2013-01-28-token-token-token/
-     * @param array $options
+     * @param  string $token   Instructions to fetch the token: https://blog.travis-ci.com/2013-01-28-token-token-token/
+     * @param  array  $options
      * @return Client
      */
     public static function create(
@@ -40,6 +49,7 @@ public static function create(
         $options = ApiSettings::getOptions($token, 'Sync', $options);
         $client = Factory::create($loop, $options);
         $asyncClient = AsyncClient::createFromClient($client);
+
         return self::createFromClient($loop, $asyncClient);
     }
 
@@ -48,25 +58,15 @@ public static function create(
      * Be sure to pass in a client with the options from ApiSettings and the Sync namespace suffix,
      * and pass in the same loop as associated with the AsyncClient you're passing in.
      *
-     * @param LoopInterface $loop
-     * @param AsyncClientInterface $asyncClient
+     * @param  LoopInterface        $loop
+     * @param  AsyncClientInterface $asyncClient
      * @return Client
      */
     public static function createFromClient(LoopInterface $loop, AsyncClientInterface $asyncClient): self
     {
         return new self($loop, $asyncClient);
     }
 
-    /**
-     * @param LoopInterface $loop
-     * @param AsyncClientInterface $asyncClient
-     */
-    private function __construct(LoopInterface $loop, AsyncClientInterface $asyncClient)
-    {
-        $this->loop = $loop;
-        $this->asyncClient = $asyncClient;
-    }
-
     /**
      * {@inheritdoc}
      */

src/ClientInterface.php

@@ -3,23 +3,17 @@
 
 namespace ApiClients\Client\Travis;
 
-use ApiClients\Foundation\Factory;
-use React\EventLoop\Factory as LoopFactory;
-use React\EventLoop\LoopInterface;
-use Rx\React\Promise;
 use ApiClients\Client\Travis\Resource\RepositoryInterface;
 use ApiClients\Client\Travis\Resource\SSHKeyInterface;
 use ApiClients\Client\Travis\Resource\UserInterface;
-use function Clue\React\Block\await;
-use function React\Promise\resolve;
 
 interface ClientInterface
 {
     /**
      * Fetch the given repository.
      * This will return a Resource\Async\Repository object.
      *
-     * @param string $repository
+     * @param  string              $repository
      * @return RepositoryInterface
      */
     public function repository(string $repository): RepositoryInterface;
@@ -29,15 +23,15 @@ public function repository(string $repository): RepositoryInterface;
      * on the hooks observable before it fetches the repository information.
      * It could be used to only fetch new repositories.
      *
-     * @param callable|null $filter
+     * @param  callable|null $filter
      * @return array
      */
     public function repositories(callable $filter = null): array;
 
     /**
      * Fetch information about the current authenticated user.
      * This will return a Resource\Sync\User object.
-     * (Requires auth token to be passed to the client!)
+     * (Requires auth token to be passed to the client!).
      *
      * @return UserInterface
      */
@@ -47,9 +41,9 @@ public function user(): UserInterface;
      * Fetch the SSH key for the given repository ID.
      * This will return a Resource\Sync\SSHKey object.
      * (Requires auth token to be passed to the client!)
-     * (Only available on Travis Pro: https://travis-ci.com/)
+     * (Only available on Travis Pro: https://travis-ci.com/).
      *
-     * @param int $id
+     * @param  int             $id
      * @return SSHKeyInterface
      */
     public function sshKey(int $id): SSHKeyInterface;
@@ -59,7 +53,7 @@ public function sshKey(int $id): SSHKeyInterface;
      * Hooks represent whether a repository is active or not,
      * their ID's match that of the repository they represent.
      * This will return an array of Resource\Sync\Hook objects.
-     * (Requires auth token to be passed to the client!)
+     * (Requires auth token to be passed to the client!).
      *
      * @return array
      */
@@ -68,7 +62,7 @@ public function hooks(): array;
     /**
      * Fetch a stream of which users and orgs the currently authenticated user has access to.
      * This will return an array of Resource\Sync\Account objects.
-     * (Requires auth token to be passed to the client!)
+     * (Requires auth token to be passed to the client!).
      *
      * @return array
      */
@@ -77,7 +71,7 @@ public function accounts(): array;
     /**
      * Fetch a list of currently active broadcast. Used by the Travis team to spread news on the site.
      * This will return an array of Resource\Sync\Broadcast objects.
-     * (Requires auth token to be passed to the client!)
+     * (Requires auth token to be passed to the client!).
      *
      * @return array
      */

src/CommandBus/Handler/AccountsHandler.php

@@ -7,7 +7,6 @@
 use ApiClients\Tools\Services\Client\FetchAndIterateService;
 use React\Promise\PromiseInterface;
 use function React\Promise\resolve;
-use function WyriHaximus\React\futureFunctionPromise;
 
 final class AccountsHandler
 {
@@ -25,9 +24,9 @@ public function __construct(FetchAndIterateService $fetchAndIterateService)
     }
 
     /**
-     * Fetch the given repository and hydrate it
+     * Fetch the given repository and hydrate it.
      *
-     * @param AccountsCommand $command
+     * @param  AccountsCommand  $command
      * @return PromiseInterface
      */
     public function handle(AccountsCommand $command): PromiseInterface

src/CommandBus/Handler/AnnotationsHandler.php

@@ -8,7 +8,6 @@
 use ApiClients\Tools\Services\Client\FetchAndIterateService;
 use React\Promise\PromiseInterface;
 use function React\Promise\resolve;
-use function WyriHaximus\React\futureFunctionPromise;
 
 final class AnnotationsHandler
 {
@@ -26,9 +25,9 @@ public function __construct(FetchAndIterateService $service)
     }
 
     /**
-     * Fetch the given repository and hydrate it
+     * Fetch the given repository and hydrate it.
      *
-     * @param AnnotationsCommand $command
+     * @param  AnnotationsCommand $command
      * @return PromiseInterface
      */
     public function handle(AnnotationsCommand $command): PromiseInterface

src/CommandBus/Handler/BranchesHandler.php

@@ -8,7 +8,6 @@
 use ApiClients\Tools\Services\Client\FetchAndIterateService;
 use React\Promise\PromiseInterface;
 use function React\Promise\resolve;
-use function WyriHaximus\React\futureFunctionPromise;
 
 final class BranchesHandler
 {
@@ -26,9 +25,9 @@ public function __construct(FetchAndIterateService $service)
     }
 
     /**
-     * Fetch the given repository and hydrate it
+     * Fetch the given repository and hydrate it.
      *
-     * @param BranchesCommand $command
+     * @param  BranchesCommand  $command
      * @return PromiseInterface
      */
     public function handle(BranchesCommand $command): PromiseInterface