Skip to content

Commit

Permalink
bug #1014 hiding php8 file so it doesn't throw autoloading warning (w…
Browse files Browse the repository at this point in the history
…eaverryan)

This PR was merged into the 1.0-dev branch.

Discussion
----------

hiding php8 file so it doesn't throw autoloading warning

Fixes #1013

The contents of this odd file need to live in a separate file so that PHP 7 can parse `MigrationDiffFilteredOutput.php` without encountering a syntax error on the `string|iterable` type. If anyone can suggest a nicer solution, I'm all ears :).

Commits
-------

583c67e hiding php8 file so it doesn't throw autoloading warning
  • Loading branch information
weaverryan committed Nov 17, 2021
2 parents 40b9b55 + 583c67e commit ccb0d55
Show file tree
Hide file tree
Showing 3 changed files with 1 addition and 2 deletions.
1 change: 0 additions & 1 deletion .php-cs-fixer.dist.php
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,6 @@
->exclude('fixtures')
// the PHP template files are a bit special
->notName('*.tpl.php')
->notPath('src/Console/MigrationDiffFilteredOutput_php8.php')
;

return (new PhpCsFixer\Config())
Expand Down
2 changes: 1 addition & 1 deletion src/Console/MigrationDiffFilteredOutput.php
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,7 @@ public function setDecorated($decorated)
}
}
} else {
require __DIR__.'/MigrationDiffFilteredOutput_php8.php';
require __DIR__.'/MigrationDiffFilteredOutput_php8';
}

trait BaseMakerMigrationDiffFilteredOuputTrait
Expand Down

0 comments on commit ccb0d55

Please sign in to comment.