Skip to content
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Laravel 9.x #36

Merged
merged 8 commits into from
Feb 9, 2022
Merged

Laravel 9.x #36

merged 8 commits into from
Feb 9, 2022

Conversation

rennokki
Copy link
Member

@rennokki rennokki commented Feb 3, 2022

This pull request includes changes from your build using the "Shift Workbench".

Before merging, you need to:

  • Checkout the shift-build-2349 branch
  • Review all comments for additional changes
  • Thoroughly test your application

Don't hesitate to send your feedback to shift@laravelshift.com or share your ❤️ for Shift on Twitter.

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
@rennokki
Copy link
Member Author

rennokki commented Feb 3, 2022

✅ Shift did not find any models which unnecessarily set the $table property.

@rennokki
Copy link
Member Author

rennokki commented Feb 3, 2022

✅ Shift did not find any unnecessary attributes listed in the $dates property of your models.

@rennokki
Copy link
Member Author

rennokki commented Feb 3, 2022

✅ Shift did not find any orderBy clauses which could be streamlined.

@codecov
Copy link

codecov bot commented Feb 3, 2022

Codecov Report

Merging #36 (82ce7cd) into master (90fe7fe) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master      #36   +/-   ##
=========================================
  Coverage     92.82%   92.82%           
  Complexity       58       58           
=========================================
  Files             5        5           
  Lines           195      195           
=========================================
  Hits            181      181           
  Misses           14       14           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 90fe7fe...82ce7cd. Read the comment docs.

@rennokki rennokki merged commit 32fc475 into master Feb 9, 2022
@rennokki rennokki deleted the shift-build-2349 branch February 9, 2022 22:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants