Code Style improvements (#92)

* Apply phpcs

* Simplify code (extra new lines and else statements)

* Remove unused methods

* Fix docblocks

* Fix inheritance instantiation bug

* Fix type cast bug

* Fix incompatible types

* Remove unused env trait

* Improve type hint

* Fix docblocks and service client codestyle

* Fix bad docblock signatures

Author: SerafimArts

src/Client/ActivityCompletionClientInterface.php

@@ -25,21 +25,13 @@ interface ActivityCompletionClientInterface
      * @param string $activityId
      * @param mixed $result
      */
-    public function complete(
-        string $workflowId,
-        ?string $runId,
-        string $activityId,
-        $result = null
-    ): void;
+    public function complete(string $workflowId, ?string $runId, string $activityId, $result = null): void;
 
     /**
      * @param string $taskToken
      * @param mixed $result
      */
-    public function completeByToken(
-        string $taskToken,
-        $result = null
-    ): void;
+    public function completeByToken(string $taskToken, $result = null): void;
 
     /**
      * @param string $workflowId
@@ -58,10 +50,7 @@ public function completeExceptionally(
      * @param string $taskToken
      * @param \Throwable $error
      */
-    public function completeExceptionallyByToken(
-        string $taskToken,
-        \Throwable $error
-    ): void;
+    public function completeExceptionallyByToken(string $taskToken, \Throwable $error): void;
 
     /**
      * @param string $workflowId
@@ -80,10 +69,7 @@ public function reportCancellation(
      * @param string $taskToken
      * @param $details
      */
-    public function reportCancellationByToken(
-        string $taskToken,
-        $details = null
-    ): void;
+    public function reportCancellationByToken(string $taskToken, $details = null): void;
 
     /**
      * @param string $workflowId
@@ -106,8 +92,5 @@ public function recordHeartbeat(
      *
      * @throw ActivityCanceledException
      */
-    public function recordHeartbeatByToken(
-        string $taskToken,
-        $details = null
-    );
+    public function recordHeartbeatByToken(string $taskToken, $details = null);
 }

src/Client/GRPC/BaseClient.php

@@ -143,7 +143,7 @@ protected function invoke(string $method, object $arg, ContextInterface $ctx = n
                 }
 
                 // wait till next call
-                usleep($waitRetry->totalMicroseconds);
+                \usleep((int)$waitRetry->totalMicroseconds);
 
                 $waitRetry = CarbonInterval::millisecond(
                     $waitRetry->totalMilliseconds + $retryOption->backoffCoefficient