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

Merged
merged 5 commits into from Feb 9, 2022
Merged

Laravel 9.x #17

merged 5 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 #17 (c301829) into master (e745962) will decrease coverage by 6.62%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##             master      #17      +/-   ##
============================================
- Coverage     95.00%   88.37%   -6.63%     
  Complexity       14       14              
============================================
  Files             5        6       +1     
  Lines            40       43       +3     
============================================
  Hits             38       38              
- Misses            2        5       +3     
Impacted Files Coverage Δ
src/Database/SchemaState.php 0.00% <0.00%> (ø)

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 e745962...c301829. Read the comment docs.

@rennokki rennokki merged commit 62a6211 into master Feb 9, 2022
@rennokki rennokki deleted the shift-build-2349 branch February 9, 2022 22:40
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