return types

Author: Litarnus

src/Monolog/LogsHandler.php

@@ -7,6 +7,7 @@
 use Monolog\Formatter\FormatterInterface;
 use Monolog\Handler\HandlerInterface;
 use Monolog\Logger as MonologLogger;
+use Monolog\LogRecord;
 use Sentry\Monolog\CompatibilityLogLevelTrait;
 use Sentry\Monolog\LogsHandler as BaseLogsHandler;
 
@@ -30,11 +31,17 @@ public function __construct(int $level = MonologLogger::DEBUG, bool $bubble = tr
         $this->logsHandler = new BaseLogsHandler($logLevel, $bubble);
     }
 
+    /**
+     * @param array<string, mixed>|LogRecord $record
+     */
     public function isHandling(array $record): bool
     {
         return $this->logsHandler->isHandling($record);
     }
 
+    /**
+     * @param array<string, mixed>|LogRecord $record
+     */
     public function handle(array $record): bool
     {
         // Extra check required here because `isHandling` is not guaranteed to
@@ -46,6 +53,9 @@ public function handle(array $record): bool
         return false;
     }
 
+    /**
+     * @param array<array<string, mixed>|LogRecord> $records
+     */
     public function handleBatch(array $records): void
     {
         $this->logsHandler->handleBatch($records);
@@ -56,6 +66,9 @@ public function close(): void
         $this->logsHandler->close();
     }
 
+    /**
+     * @param callable $callback
+     */
     public function pushProcessor($callback): void
     {
         $this->logsHandler->pushProcessor($callback);

tests/End2End/App/Command/CrashCommand.php

@@ -29,7 +29,7 @@ public function __construct(LoggerInterface $logger, string $subcommand = null)
         $this->subcommand = $subcommand;
     }
 
-    protected function execute(InputInterface $input, OutputInterface $output)
+    protected function execute(InputInterface $input, OutputInterface $output): int
     {
         $this->logger->warning('Executing subcommand if exists');
 

tests/End2End/App/Command/LoggingCommand.php

@@ -22,7 +22,7 @@ public function __construct(LoggerInterface $logger)
         $this->logger = $logger;
     }
 
-    protected function execute(InputInterface $input, OutputInterface $output)
+    protected function execute(InputInterface $input, OutputInterface $output): int
     {
         $this->logger->debug('Debug Log');
 

tests/End2End/LoggingCommandTest.php

@@ -98,6 +98,11 @@ public function testExceptionLogsWithSubcommand(): void
         $this->assertEmpty($logEvent->getExceptions());
     }
 
+    /**
+     * @param Log[] $logs
+     * @param LogLevel $level
+     * @return Log|null
+     */
     private function findOneByLevel(array $logs, LogLevel $level): ?Log
     {
         foreach ($logs as $log) {