Skip to content

[3.7] Laravel 7 support #2097

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

Merged
merged 4 commits into from
Sep 14, 2020
Merged

[3.7] Laravel 7 support #2097

merged 4 commits into from
Sep 14, 2020

Conversation

divine
Copy link
Contributor

@divine divine commented Sep 14, 2020

  • Added mongodb 4.4 to matrix
  • Updated dependencies
  • Removed coveralls due to package php > 7.3 constraints
  • Removed CacheTest which doesn't exist?
  • Updated tests

For all changes please check https://laravel.com/docs/7.x/upgrade

- Added mongodb 4.4 to matrix
- Updated dependencies
- Removed coveralls due to package  php > 7.3 contstraints
- Removed CacheTest which doesn't exist?
- Updated tests

For all changes please check https://laravel.com/docs/7.x/upgrade
@divine divine requested a review from Smolevich September 14, 2020 10:06
@codecov-commenter
Copy link

codecov-commenter commented Sep 14, 2020

Codecov Report

Merging #2097 into master will increase coverage by 0.01%.
The diff coverage is 93.75%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master    #2097      +/-   ##
============================================
+ Coverage     86.88%   86.90%   +0.01%     
+ Complexity      671      664       -7     
============================================
  Files            31       33       +2     
  Lines          1556     1573      +17     
============================================
+ Hits           1352     1367      +15     
- Misses          204      206       +2     
Impacted Files Coverage Δ Complexity Δ
src/Jenssegers/Mongodb/Eloquent/Model.php 94.90% <90.90%> (+0.03%) 83.00 <10.00> (ø)
src/Jenssegers/Mongodb/Query/Builder.php 89.26% <100.00%> (+0.20%) 163.00 <0.00> (-7.00) ⬆️
...rs/Mongodb/Queue/Failed/MongoFailedJobProvider.php 21.05% <0.00%> (-1.17%) 5.00% <0.00%> (ø%)
src/Jenssegers/Mongodb/Eloquent/SoftDeletes.php 100.00% <0.00%> (ø) 1.00% <0.00%> (ø%)
src/Jenssegers/Mongodb/MongodbServiceProvider.php 100.00% <0.00%> (ø) 2.00% <0.00%> (ø%)
...s/Mongodb/Validation/ValidationServiceProvider.php 100.00% <0.00%> (ø) 1.00% <0.00%> (ø%)
src/Jenssegers/Mongodb/Helpers/EloquentBuilder.php 100.00% <0.00%> (ø) 0.00% <0.00%> (?%)
src/Jenssegers/Mongodb/Auth/User.php 0.00% <0.00%> (ø) 0.00% <0.00%> (?%)
src/Jenssegers/Mongodb/Relations/BelongsToMany.php 86.00% <0.00%> (+0.14%) 42.00% <0.00%> (ø%)
... and 5 more

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 32ff280...0804ab2. Read the comment docs.

Remove phpcov, it's not compatible and suggested to be installed with phar instead of composer, see sebastianbergmann/phpcov#96
Remove shouldUseCollections function which checks Laravel version > 5.3, it was introduced to support version 5.3 in #925
@Smolevich Smolevich merged commit dd6f667 into master Sep 14, 2020
@divine divine deleted the l7 branch September 14, 2020 12:28
@divine divine changed the title Laravel 7 support [3.7] Laravel 7 support Sep 18, 2020
mnphpexpert added a commit to mnphpexpert/laravel-mongodb that referenced this pull request Sep 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants