Migrate php71 to pre-php80 #387
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Update composer.json PHP version. - Create rector.php - Set migration to 7.2 - Exclude AddClosureVoidReturnTypeWhereNoReturnRector rule. This is a NOOP.
- Update composer.json PHP version. - Set migration to 7.3 - Migrate - Add log file with results of migration
- Update composer.json PHP version. - Set migration to 7.4 - Migrate - Add log file with results of migration This was a noop.
- Update packages installed by composer. It should be noted that the following packages are abandoned (and their recommended replacements, if any):. Package jakub-onderka/php-console-color is abandoned, you should avoid using it. Use php-parallel-lint/php-console-color instead. Package jakub-onderka/php-console-highlighter is abandoned, you should avoid using it. Use php-parallel-lint/php-console-highlighter instead. Package jenssegers/mongodb is abandoned, you should avoid using it. Use mongodb/laravel-mongodb instead. Package laravelcollective/html is abandoned, you should avoid using it. Use spatie/laravel-html instead. Package swiftmailer/swiftmailer is abandoned, you should avoid using it. Use symfony/mailer instead. Package symfony/debug is abandoned, you should avoid using it. Use symfony/error-handler instead. Package themsaid/laravel-routes-publisher is abandoned, you should avoid using it. No replacement was suggested. Package zendframework/zend-diactoros is abandoned, you should avoid using it. Use laminas/laminas-diactoros instead. Package fzaninotto/faker is abandoned, you should avoid using it. No replacement was suggested. Package phpunit/php-token-stream is abandoned, you should avoid using it. No replacement was suggested. Package sebastian/finder-facade is abandoned, you should avoid using it. No replacement was suggested. Package theseer/fdomdocument is abandoned, you should avoid using it. No replacement was suggested.
- Require driftingly/rector-laravel for dev environment only. - Composer update - NPM update(depreciated list in package-lock.json)
- Cleanup the rector.log prior to merge to establish a pattern to avoid merge conflicts.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Several migrations moving from PHP 7.1.3 to PHP 7.4 just prior to some major composer requirement changes in preparation for PHP 8.0. Many commits include a rector.log file showing the output from the rector run.