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 #33

Merged
merged 7 commits into from
Feb 10, 2022
Merged

Laravel 9.x #33

merged 7 commits into from
Feb 10, 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 #33 (ab1ca15) into master (837fc41) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master      #33   +/-   ##
=========================================
  Coverage     93.42%   93.42%           
  Complexity       22       22           
=========================================
  Files             3        3           
  Lines            76       76           
=========================================
  Hits             71       71           
  Misses            5        5           

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 837fc41...ab1ca15. Read the comment docs.

@rennokki rennokki merged commit 85f45c3 into master Feb 10, 2022
@rennokki rennokki deleted the shift-build-2349 branch February 10, 2022 01:19
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.

2 participants