- Notifications
You must be signed in to change notification settings - Fork45
Laravel 9.x Shift#319
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
base:main
Are you sure you want to change the base?
Uh oh!
There was an error while loading.Please reload this page.
Conversation
Laravel 8 adopts the tuple syntax for controller actions. Since the old options array is incompatible with this syntax, Shift converted them to use modern, fluent methods.
In an effort to make upgrading the constantly changing config fileseasier, Shift defaulted them and merged your true customizations -where ENV variables may not be used.
`<env>` tags have a lower precedence than system environment variables making it easier to overwrite PHPUnit configuration values in additional environments, such a CI.Review this blog post for more details on configuration precedence when testing Laravel:https://jasonmccreary.me/articles/laravel-testing-configuration-precedence/
From the [PHPUnit 8 release notes][1], the `TestCase` methods below now declare a `void` return type:- `setUpBeforeClass()`- `setUp()`- `assertPreConditions()`- `assertPostConditions()`- `tearDown()`- `tearDownAfterClass()`- `onNotSuccessfulTest()`[1]:https://phpunit.de/announcements/phpunit-8.html
vaughnjones-marcus commentedOct 4, 2023
ℹ️ Shift noticed you have additional namespaces in your application. You may use theConsolidate Namespaces Shift to simplify your namespaces into the default Laravel |
vaughnjones-marcus commentedOct 4, 2023
ℹ️ Laravel 9 moved the |
vaughnjones-marcus commentedOct 4, 2023
Dependencies composer require symfony/mailgun-mailer symfony/http-client For more details on what has changed you may review theSymfony Mailer section of theUpgrade Guide. |
vaughnjones-marcus commentedOct 4, 2023
|
vaughnjones-marcus commentedOct 4, 2023
You shouldreview this commit for additional customizations or opportunities to use new |
vaughnjones-marcus commentedOct 4, 2023
ℹ️ The |
vaughnjones-marcus commentedOct 4, 2023
If you were using the |
vaughnjones-marcus commentedOct 4, 2023
ℹ️ Shiftupdated your dependencies for Laravel 9. While many of the popular packages are reviewed, you may have to update additional packages in order for your application to be compatible with Laravel 9. Watchdealing with dependencies for tips on handling any Composer issues. |
vaughnjones-marcus commentedOct 4, 2023
ℹ️ Laravel 9 adoptedanonymous migrations. Shiftautomated this change to align with modern Laravel conventions and avoid naming migrations. |
vaughnjones-marcus commentedOct 4, 2023
ℹ️ All of the underlying Symfony components used by Laravel have been upgraded to Symfony 6. If you are directly interacting with any Symfony component, you should review theSymfony change log for additional changes. |
vaughnjones-marcus commentedOct 4, 2023
ℹ️ The HTTP client has been updated to have a default timeout of 30 seconds. Previously there was no default. The underlying request would wait indefinitely. Now an exception is thrown after 30 seconds. If you find your requests need longer, you may prefix your request to specify a timeout: Http::timeout(60)->get("https://laravelshift.com"); |
vaughnjones-marcus commentedOct 4, 2023
ℹ️ Laravel 9 now uses Vite to build frontend assets. While you may continue to use Laravel Mix, it is no longer the default. If you wish to modernize your application to use Vite, you may run theVite Converter for free. |
This pull request includes the changes for upgrading to Laravel 9.x. Feel free to commit any additional changes to the
shift-101902branch.Before merging, you need to:
shift-101902branchcomposer update(if the scripts fail, add--no-scripts)If you need help with your upgrade, check out theHuman Shifts. You may also join theShifty Coders Slack workspace to level-up your Laravel skills.