fix code style

Author: Kate Simozhenko

.github/workflows/ci.yml

@@ -29,6 +29,7 @@ jobs:
 
       - run: sed -i 's/525568/16777471/' vendor/kwn/php-rdkafka-stubs/stubs/constants.php
 
+      - run: cd docker && docker build --rm --force-rm --no-cache --pull --tag "enqueue/dev:latest" -f Dockerfile .
       - run: docker run --workdir="/mqdev" -v "`pwd`:/mqdev" --rm enqueue/dev:latest php -d memory_limit=1024M bin/phpstan analyse -l 1 -c phpstan.neon --error-format=github -- ${{ env.GIT_DIFF_FILTERED }}
         if: env.GIT_DIFF_FILTERED
 

pkg/sns/SnsDestination.php

@@ -77,8 +77,6 @@ public function getDeliveryPolicy(): ?int
      * You can't change it for an existing topic. When you set this attribute, you must provide aMessageGroupId
      * explicitly.
      * For more information, see https://docs.aws.amazon.com/sns/latest/dg/sns-fifo-topics.html
-     *
-     * @param bool $enable
      */
     public function setFifoTopic(bool $enable): void
     {

pkg/sns/SnsMessage.php

@@ -58,15 +58,6 @@ class SnsMessage implements Message
      * See AWS documentation for message attribute structure.
      *
      * @see https://docs.aws.amazon.com/aws-sdk-php/v3/api/api-sns-2010-03-31.html#shape-messageattributevalue
-     *
-     * @param string      $body
-     * @param array       $properties
-     * @param array       $headers
-     * @param array|null  $messageAttributes
-     * @param string|null $messageStructure
-     * @param string|null $phoneNumber
-     * @param string|null $subject
-     * @param string|null $targetArn
      */
     public function __construct(
         string $body = '',
@@ -89,89 +80,58 @@ public function __construct(
         $this->redelivered = false;
     }
 
-    /**
-     * @return string|null
-     */
     public function getSnsMessageId(): ?string
     {
         return $this->snsMessageId;
     }
 
-    /**
-     * @param string|null $snsMessageId
-     */
     public function setSnsMessageId(?string $snsMessageId): void
     {
         $this->snsMessageId = $snsMessageId;
     }
 
-    /**
-     * @return string|null
-     */
     public function getMessageStructure(): ?string
     {
         return $this->messageStructure;
     }
 
-    /**
-     * @param string|null $messageStructure
-     */
     public function setMessageStructure(?string $messageStructure): void
     {
         $this->messageStructure = $messageStructure;
     }
 
-    /**
-     * @return string|null
-     */
     public function getPhoneNumber(): ?string
     {
         return $this->phoneNumber;
     }
 
-    /**
-     * @param string|null $phoneNumber
-     */
     public function setPhoneNumber(?string $phoneNumber): void
     {
         $this->phoneNumber = $phoneNumber;
     }
 
-    /**
-     * @return string|null
-     */
     public function getSubject(): ?string
     {
         return $this->subject;
     }
 
-    /**
-     * @param string|null $subject
-     */
     public function setSubject(?string $subject): void
     {
         $this->subject = $subject;
     }
 
-    /**
-     * @return array|null
-     */
     public function getMessageAttributes(): ?array
     {
         return $this->messageAttributes;
     }
 
-    /**
-     * @param array|null $messageAttributes
-     */
     public function setMessageAttributes(?array $messageAttributes): void
     {
         $this->messageAttributes = $messageAttributes;
     }
 
     /**
-     * @param string $name
-     * @param null   $default
+     * @param null $default
      *
      * @return array|null
      */
@@ -187,9 +147,6 @@ public function getAttribute(string $name, $default = null)
      *        'DataType' => '<string>', // REQUIRED
      *        'StringValue' => '<string>',
      *     ].
-     *
-     * @param string     $name
-     * @param array|null $attribute
      */
     public function setAttribute(string $name, ?array $attribute): void
     {
@@ -201,7 +158,6 @@ public function setAttribute(string $name, ?array $attribute): void
     }
 
     /**
-     * @param string                          $name
      * @param string                          $dataType String, String.Array, Number, or Binary
      * @param string|resource|StreamInterface $value
      */
@@ -215,17 +171,11 @@ public function addAttribute(string $name, string $dataType, $value): void
         ];
     }
 
-    /**
-     * @return string|null
-     */
     public function getTargetArn(): ?string
     {
         return $this->targetArn;
     }
 
-    /**
-     * @param string|null $targetArn
-     */
     public function setTargetArn(?string $targetArn): void
     {
         $this->targetArn = $targetArn;
@@ -241,17 +191,12 @@ public function setTargetArn(?string $targetArn): void
      * data for multiple users). In this scenario, multiple readers can process the queue, but the session data
      * of each user is processed in a FIFO fashion.
      * For more information, see: https://docs.aws.amazon.com/sns/latest/dg/fifo-message-grouping.html
-     *
-     * @param string|null $id
      */
     public function setMessageGroupId(string $id = null): void
     {
         $this->messageGroupId = $id;
     }
 
-    /**
-     * @return string|null
-     */
     public function getMessageGroupId(): ?string
     {
         return $this->messageGroupId;