chore: Remove some TODOs (#286)

Author: theofidry

src/ChunkedItemsIterator.php

@@ -141,7 +141,6 @@ private static function normalizeItemStream(mixed $items): Iterator
             $items,
             sprintf(
                 'Expected the fetched items to be a list or an iterable of strings. Got "%s".',
-                // TODO: use get_debug_type when dropping PHP 7.4 support
                 get_debug_type($items),
             ),
         );
@@ -164,7 +163,6 @@ private static function normalizeItem(mixed $item, int|string $index): string
             $item,
             sprintf(
                 'The items are potentially passed to the child processes via the STDIN. For this reason they are expected to be string values. Got "%s" for the item "%s".',
-                // TODO: use get_debug_type when dropping PHP 7.4 support
                 get_debug_type($item),
                 $index,
             ),

src/Input/InputOptionsSerializer.php

@@ -22,7 +22,6 @@
 use function array_map;
 use function implode;
 use function is_string;
-use function method_exists;
 use function preg_match;
 use function sprintf;
 use function str_replace;
@@ -71,8 +70,7 @@ private static function serializeOption(
         string $name,
         array|bool|float|int|string|null $value,
     ): string {
-        // TODO: remove the method exists check once we drop support for Symfony 4.4
-        if (method_exists(InputOption::class, 'isNegatable') && $option->isNegatable()) {
+        if ($option->isNegatable()) {
             return sprintf(
                 '--%s%s',
                 $value ? '' : 'no-',

src/Process/StandardSymfonyProcessFactory.php

@@ -15,7 +15,6 @@
 
 use Symfony\Component\Process\InputStream;
 use Symfony\Component\Process\Process;
-use function method_exists;
 
 final class StandardSymfonyProcessFactory implements SymfonyProcessFactory
 {
@@ -36,12 +35,7 @@ public function startProcess(
         );
 
         $process->setInput($inputStream);
-        // TODO: remove the following once dropping Symfony 4.4. Environment
-        //  variables are always inherited as of 5.0
         // @codeCoverageIgnoreStart
-        if (method_exists($process, 'inheritEnvironmentVariables')) {
-            $process->inheritEnvironmentVariables(true);
-        }
         $process->start(
             static fn (string $type, string $buffer) => $processOutput(
                 $index,