Merge pull request #322 from laravel/fix/removes-warning

Removes warning
laravel · Dec 30, 2024 · 6857eef · 6857eef
2 parents 6945563 + 5a62e34
commit 6857eef
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 13 deletions.
diff --git a/.github/workflows/tests.yml b/.github/workflows/tests.yml
@@ -50,11 +50,6 @@ jobs:
           composer install
 
       - name: Execute lint tests with Laravel preset
-        if: matrix.php == '8.4'
-        run: PINT_IGNORE_ENV=1 ./pint --test
-
-      - name: Execute lint tests with Laravel preset
-        if: matrix.php != '8.4'
         run: ./pint --test
 
       - name: Execute static analysis

diff --git a/app/Commands/DefaultCommand.php b/app/Commands/DefaultCommand.php
@@ -57,14 +57,6 @@ protected function configure()
      */
     public function handle($fixCode, $elaborateSummary)
     {
-        if (! ($_ENV['PINT_IGNORE_ENV'] ?? false) && version_compare(PHP_VERSION, '8.4.0', '>=')) {
-            $this->warn('PHP 8.4 is not currently supported by PHP CS Fixer and you may encounter unexpected behavior.');
-            $this->warn('You can still force Pint to run by setting the environment variable:');
-            $this->warn('PINT_IGNORE_ENV=1 vendor/bin/pint');
-
-            return 1;
-        }
-
         [$totalFiles, $changes] = $fixCode->execute();
 
         return $elaborateSummary->execute($totalFiles, $changes);