feat: add no_superflous_phpdoc_tags rule to pint config

Author: calebdw

pint.json

@@ -98,6 +98,7 @@
             ]
         },
         "no_spaces_after_function_name": true,
+        "no_superfluous_phpdoc_tags": true,
         "no_trailing_comma_in_singleline": true,
         "no_trailing_whitespace": true,
         "no_trailing_whitespace_in_comment": true,

src/Illuminate/Console/Scheduling/Schedule.php

@@ -301,7 +301,6 @@ public function exec($command, array $parameters = [])
     /**
      * Create new schedule group.
      *
-     * @param  \Illuminate\Console\Scheduling\Event  $event
      * @return void
      *
      * @throws \RuntimeException

src/Illuminate/Container/Container.php

@@ -677,7 +677,7 @@ protected function removeAbstractAlias($searched)
      * Assign a set of tags to a given binding.
      *
      * @param  array|string  $abstracts
-     * @param  mixed  ...$tags
+     * @param  list<string>|string  ...$tags
      * @return void
      */
     public function tag($abstracts, $tags)

src/Illuminate/Contracts/Container/Container.php

@@ -40,7 +40,7 @@ public function alias($abstract, $alias);
      * Assign a set of tags to a given binding.
      *
      * @param  array|string  $abstracts
-     * @param  mixed  ...$tags
+     * @param  list<string>|string  ...$tags
      * @return void
      */
     public function tag($abstracts, $tags);

src/Illuminate/Foundation/helpers.php

@@ -500,9 +500,9 @@ function encrypt($value, $serialize = true): string
      * @param  bool  $halt
      * @return array|null
      */
-    function event(...$args)
+    function event($event, $payload = [], $halt = false)
     {
-        return app('events')->dispatch(...$args);
+        return app('events')->dispatch($event, $payload, $halt);
     }
 }
 

src/Illuminate/Queue/Events/WorkerStarting.php

@@ -9,7 +9,7 @@ class WorkerStarting
      *
      * @param  string  $connectionName
      * @param  string  $queue
-     * @param  \Illuminate\Queue\WorkerOptions  $options
+     * @param  \Illuminate\Queue\WorkerOptions  $workerOptions
      */
     public function __construct(
         public $connectionName,

src/Illuminate/Support/Testing/Fakes/BusFake.php

@@ -137,7 +137,6 @@ public function assertDispatched($command, $callback = null)
      * Assert if a job was pushed exactly once.
      *
      * @param  string|\Closure  $command
-     * @param  int  $times
      * @return void
      */
     public function assertDispatchedOnce($command)

src/Illuminate/Support/Testing/Fakes/EventFake.php

@@ -151,7 +151,6 @@ public function assertDispatched($event, $callback = null)
      * Assert if an event was dispatched exactly once.
      *
      * @param  string  $event
-     * @param  int  $times
      * @return void
      */
     public function assertDispatchedOnce($event)

tests/Database/PruneCommandTest.php

@@ -242,7 +242,7 @@ public function testTheCommandDispatchesEvents()
                 $event->models === [Pruning\Models\PrunableTestModelWithPrunableRecords::class];
         });
         $dispatcher->shouldReceive('listen')->once()->with(ModelsPruned::class, m::type(Closure::class));
-        $dispatcher->shouldReceive('dispatch')->twice()->with(m::type(ModelsPruned::class));
+        $dispatcher->shouldReceive('dispatch')->twice()->with(m::type(ModelsPruned::class), [], false);
         $dispatcher->shouldReceive('dispatch')->once()->withArgs(function ($event) {
             return get_class($event) === ModelPruningFinished::class &&
                 $event->models === [Pruning\Models\PrunableTestModelWithPrunableRecords::class];

tests/Integration/Database/EloquentMassPrunableTest.php

@@ -57,7 +57,7 @@ public function testPrunesRecords()
         app('events')
             ->shouldReceive('dispatch')
             ->times(2)
-            ->with(m::type(ModelsPruned::class));
+            ->with(m::type(ModelsPruned::class), [], false);
 
         collect(range(1, 5000))->map(function ($id) {
             return ['name' => 'foo'];
@@ -76,7 +76,7 @@ public function testPrunesSoftDeletedRecords()
         app('events')
             ->shouldReceive('dispatch')
             ->times(3)
-            ->with(m::type(ModelsPruned::class));
+            ->with(m::type(ModelsPruned::class), [], false);
 
         collect(range(1, 5000))->map(function ($id) {
             return ['deleted_at' => now()];