Fix PHPStan warnings.

phpspec.yml.dist

@@ -1,6 +1,5 @@
 formatter.name: pretty
 extensions:
-  drupol\PhpspecAnnotation\PhpspecAnnotation: ~
   LeanPHP\PhpSpec\CodeCoverage\CodeCoverageExtension:
     format:
       - html

src/Base.php

@@ -11,6 +11,7 @@
 use drupol\collection\Iterator\ClosureIterator;
 use drupol\collection\Transformation\Run;
 use drupol\collection\Transformation\Transform;
+use Generator;
 
 use function is_string;
 
@@ -30,15 +31,15 @@ abstract class Base implements BaseInterface
      * @param Closure|iterable|mixed $data
      * @param mixed ...$parameters
      */
-    public function __construct($data = [], ...$parameters)
+    final public function __construct($data = [], ...$parameters)
     {
         switch (true) {
             case $data instanceof Closure:
                 $this->source = $data;
 
                 break;
             case is_iterable($data):
-                $this->source = static function () use ($data) {
+                $this->source = static function () use ($data): Generator {
                     foreach ($data as $key => $value) {
                         yield $key => $value;
                     }
@@ -49,7 +50,7 @@ public function __construct($data = [], ...$parameters)
                 $parameters += [0 => null];
                 $separator = (string) $parameters[0];
 
-                $this->source = static function () use ($data, $separator) {
+                $this->source = static function () use ($data, $separator): Generator {
                     $offset = 0;
 
                     $nextOffset = '' !== $separator ?
@@ -73,7 +74,7 @@ public function __construct($data = [], ...$parameters)
                 break;
 
             default:
-                $this->source = static function () use ($data) {
+                $this->source = static function () use ($data): Generator {
                     foreach ((array) $data as $key => $value) {
                         yield $key => $value;
                     }
@@ -82,7 +83,7 @@ public function __construct($data = [], ...$parameters)
     }
 
     /**
-     * {@inheritdoc}
+     * @return ClosureIterator<mixed>
      */
     public function getIterator(): ClosureIterator
     {

src/Collection.php

@@ -46,6 +46,7 @@
 use drupol\collection\Transformation\Implode;
 use drupol\collection\Transformation\Last;
 use drupol\collection\Transformation\Reduce;
+use Generator;
 
 use const INF;
 use const PHP_INT_MAX;
@@ -251,7 +252,7 @@ public function intersperse($element, int $every = 1, int $startAt = 0): BaseInt
     public static function iterate(callable $callback, ...$parameters): CollectionInterface
     {
         return new Collection(
-            static function () use ($parameters, $callback) {
+            static function () use ($parameters, $callback): Generator {
                 while (true) {
                     $parameters = $callback(...$parameters);
 
@@ -416,7 +417,7 @@ public function reduction(callable $callback, $initial = null): BaseInterface
      */
     public function rsample($probability): BaseInterface
     {
-        $callback = static function ($item) use ($probability) {
+        $callback = static function ($item) use ($probability): bool {
             return (mt_rand() / mt_getrandmax()) < $probability;
         };
 
@@ -488,7 +489,7 @@ public static function times($number, ?callable $callback = null): CollectionInt
         }
 
         $instance = new Collection(
-            static function () use ($number) {
+            static function () use ($number): Generator {
                 for ($current = 1; $current <= $number; ++$current) {
                     yield $current;
                 }
@@ -519,10 +520,10 @@ public function walk(callable ...$callbacks): BaseInterface
     }
 
     /**
-     * @param array $data
+     * @param array<mixed> $data
      * @param mixed ...$parameters
      *
-     * @return \drupol\collection\Contract\Base
+     * @return \drupol\collection\Contract\Base<mixed>
      */
     public static function with($data = [], ...$parameters): BaseInterface
     {

src/Contract/Allable.php

@@ -12,7 +12,7 @@ interface Allable
     /**
      * Get all items from the collection.
      *
-     * @return array
+     * @return array<mixed>
      *   An array containing all the elements of the collection.
      */
     public function all(): array;

src/Contract/Appendable.php

@@ -14,7 +14,7 @@ interface Appendable
      *
      * @param mixed ...$items
      *
-     * @return \drupol\collection\Contract\Collection
+     * @return \drupol\collection\Contract\Collection<mixed>
      */
     public function append(...$items): Base;
 }

src/Contract/Applyable.php

@@ -14,7 +14,7 @@ interface Applyable
      *
      * @param callable ...$callables
      *
-     * @return \drupol\collection\Contract\Collection
+     * @return \drupol\collection\Contract\Collection<mixed>
      */
     public function apply(callable ...$callables): Base;
 }

src/Contract/Base.php

@@ -8,6 +8,8 @@
 
 /**
  * Interface Base.
+ *
+ * @template-extends IteratorAggregate<mixed>
  */
 interface Base extends IteratorAggregate, Runable, Transformable
 {

src/Contract/Chunkable.php

@@ -14,7 +14,7 @@ interface Chunkable
      *
      * @param int $size
      *
-     * @return \drupol\collection\Contract\Collection
+     * @return \drupol\collection\Contract\Collection<mixed>
      */
     public function chunk(int $size): Base;
 }

src/Contract/Collapseable.php

@@ -12,7 +12,7 @@ interface Collapseable
     /**
      * Collapse the collection of items into a single array.
      *
-     * @return \drupol\collection\Contract\Collection
+     * @return \drupol\collection\Contract\Collection<mixed>
      */
     public function collapse(): Base;
 }

src/Contract/Collection.php

@@ -55,14 +55,16 @@ interface Collection extends
 {
     /**
      * Create a new instance with no items.
+     *
+     * @return \drupol\collection\Contract\Collection<mixed>
      */
     public static function empty(): Collection;
 
     /**
      * @param callable $callback
      * @param mixed ...$parameters
      *
-     * @return \drupol\collection\Contract\Collection
+     * @return \drupol\collection\Contract\Collection<mixed>
      */
     public static function iterate(callable $callback, ...$parameters): Collection;
 
@@ -73,7 +75,7 @@ public static function iterate(callable $callback, ...$parameters): Collection;
      * @param float|int $end
      * @param int $step
      *
-     * @return \drupol\collection\Contract\Collection
+     * @return \drupol\collection\Contract\Collection<mixed>
      */
     public static function range(int $start = 0, $end = INF, $step = 1): Collection;
 
@@ -83,7 +85,7 @@ public static function range(int $start = 0, $end = INF, $step = 1): Collection;
      * @param int $number
      * @param callable $callback
      *
-     * @return \drupol\collection\Contract\Collection
+     * @return \drupol\collection\Contract\Collection<mixed>
      */
     public static function times($number, ?callable $callback = null): Collection;
 }

src/Contract/Combineable.php

@@ -12,7 +12,7 @@ interface Combineable
     /**
      * @param mixed $keys
      *
-     * @return \drupol\collection\Contract\Collection
+     * @return \drupol\collection\Contract\Collection<mixed>
      */
     public function combine($keys): Base;
 }

src/Contract/Cycleable.php

@@ -12,7 +12,7 @@ interface Cycleable
     /**
      * @param int $count
      *
-     * @return \drupol\collection\Contract\Collection
+     * @return \drupol\collection\Contract\Collection<mixed>
      */
     public function cycle(int $count = 0): Base;
 }

src/Contract/Distinctable.php

@@ -10,7 +10,7 @@
 interface Distinctable
 {
     /**
-     * @return \drupol\collection\Contract\Base
+     * @return \drupol\collection\Contract\Base<mixed>
      */
     public function distinct(): Base;
 }

src/Contract/Explodeable.php

@@ -12,7 +12,7 @@ interface Explodeable
     /**
      * @param string ...$explodes
      *
-     * @return \drupol\collection\Contract\Collection
+     * @return \drupol\collection\Contract\Collection<mixed>
      */
     public function explode(string ...$explodes): Base;
 }

src/Contract/Filterable.php

@@ -14,7 +14,7 @@ interface Filterable
      *
      * @param callable ...$callbacks
      *
-     * @return \drupol\collection\Contract\Collection
+     * @return \drupol\collection\Contract\Collection<mixed>
      */
     public function filter(callable ...$callbacks): Base;
 }

src/Contract/Flattenable.php

@@ -16,7 +16,7 @@ interface Flattenable
      *
      * @param int $depth
      *
-     * @return \drupol\collection\Contract\Collection
+     * @return \drupol\collection\Contract\Collection<mixed>
      */
     public function flatten(int $depth = PHP_INT_MAX): Base;
 }

src/Contract/Flipable.php

@@ -12,7 +12,7 @@ interface Flipable
     /**
      * Flip the items in the collection.
      *
-     * @return \drupol\collection\Contract\Collection
+     * @return \drupol\collection\Contract\Collection<mixed>
      */
     public function flip(): Base;
 }

src/Contract/Forgetable.php

@@ -14,7 +14,7 @@ interface Forgetable
      *
      * @param string ...$keys
      *
-     * @return \drupol\collection\Contract\Collection
+     * @return \drupol\collection\Contract\Collection<mixed>
      */
     public function forget(...$keys): Base;
 }

src/Contract/Intersperseable.php

@@ -17,7 +17,7 @@ interface Intersperseable
      * @param int $every
      * @param int $startAt
      *
-     * @return \drupol\collection\Contract\Collection
+     * @return \drupol\collection\Contract\Collection<mixed>
      */
     public function intersperse($element, int $every = 1, int $startAt = 0): Base;
 }

src/Contract/Keysable.php

@@ -12,7 +12,7 @@ interface Keysable
     /**
      * Get the keys of the items.
      *
-     * @return \drupol\collection\Contract\Collection
+     * @return \drupol\collection\Contract\Collection<mixed>
      */
     public function keys(): Base;
 }

src/Contract/Limitable.php

@@ -14,7 +14,7 @@ interface Limitable
      *
      * @param int $limit
      *
-     * @return \drupol\collection\Contract\Collection
+     * @return \drupol\collection\Contract\Collection<mixed>
      */
     public function limit(int $limit): Base;
 }

src/Contract/Mapable.php

@@ -14,7 +14,7 @@ interface Mapable
      *
      * @param callable ...$callbacks
      *
-     * @return \drupol\collection\Contract\Collection
+     * @return \drupol\collection\Contract\Collection<mixed>
      */
     public function map(callable ...$callbacks): Base;
 }

src/Contract/Mergeable.php

@@ -14,7 +14,7 @@ interface Mergeable
      *
      * @param iterable ...$sources
      *
-     * @return \drupol\collection\Contract\Collection
+     * @return \drupol\collection\Contract\Collection<mixed>
      */
     public function merge(...$sources): Base;
 }

src/Contract/Normalizeable.php

@@ -12,7 +12,7 @@ interface Normalizeable
     /**
      * Reset the keys on the underlying array.
      *
-     * @return \drupol\collection\Contract\Collection
+     * @return \drupol\collection\Contract\Collection<mixed>
      */
     public function normalize(): Base;
 }

src/Contract/Nthable.php

@@ -15,7 +15,7 @@ interface Nthable
      * @param int $step
      * @param int $offset
      *
-     * @return \drupol\collection\Contract\Collection
+     * @return \drupol\collection\Contract\Collection<mixed>
      */
     public function nth(int $step, int $offset = 0): Base;
 }

src/Contract/Onlyable.php

@@ -14,7 +14,7 @@ interface Onlyable
      *
      * @param mixed ...$keys
      *
-     * @return \drupol\collection\Contract\Collection
+     * @return \drupol\collection\Contract\Collection<mixed>
      */
     public function only(...$keys): Base;
 }

src/Contract/Operation.php

@@ -12,7 +12,7 @@
 interface Operation extends Transformation
 {
     /**
-     * @param iterable $collection
+     * @param iterable<mixed> $collection
      *
      * @return Closure
      */

src/Contract/Padable.php

@@ -15,7 +15,7 @@ interface Padable
      * @param int $size
      * @param mixed $value
      *
-     * @return \drupol\collection\Contract\Collection
+     * @return \drupol\collection\Contract\Collection<mixed>
      */
     public function pad(int $size, $value): Base;
 }

src/Contract/Pluckable.php

@@ -10,10 +10,10 @@
 interface Pluckable
 {
     /**
-     * @param array|string $pluck
+     * @param array<int, string>|string $pluck
      * @param mixed|null $default
      *
-     * @return \drupol\collection\Contract\Collection
+     * @return \drupol\collection\Contract\Collection<mixed>
      */
     public function pluck($pluck, $default = null): Base;
 }

src/Contract/Prependable.php

@@ -14,7 +14,7 @@ interface Prependable
      *
      * @param mixed ...$items
      *
-     * @return \drupol\collection\Contract\Collection
+     * @return \drupol\collection\Contract\Collection<mixed>
      */
     public function prepend(...$items): Base;
 }

src/Contract/RSampleable.php

@@ -12,7 +12,7 @@ interface RSampleable
     /**
      * @param float $probability
      *
-     * @return \drupol\collection\Contract\Base
+     * @return \drupol\collection\Contract\Base<mixed>
      */
     public function rsample($probability): Base;
 }

src/Contract/Rebaseable.php

@@ -10,7 +10,7 @@
 interface Rebaseable
 {
     /**
-     * @return \drupol\collection\Contract\Collection
+     * @return \drupol\collection\Contract\Collection<mixed>
      */
     public function rebase(): Base;
 }