laravel-doctrine / orm

An integration library for Laravel and Doctrine ORM
MIT License
829 stars 177 forks source link

[FEATURE] Doctrine ORM 3.0 & DBAL 4.0 & Doctrine Coding Standard #582

Closed TomHAnderson closed 1 month ago

TomHAnderson commented 1 month ago

Please prefix your pull request with one of the following: [FIX] [FEATURE].

Changes proposed in this pull request:

TomHAnderson commented 1 month ago

@eigan Before I do anything within this organization, will you please email me? tom.h.anderson@gmail.com

codecov-commenter commented 1 month ago

:warning: Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

Attention: Patch coverage is 63.33333% with 231 lines in your changes missing coverage. Please review.

Please upload report for BASE (3.0.x@ee9f2e8). Learn more about missing BASE report.

Files with missing lines Patch % Lines
src/DoctrineServiceProvider.php 0.00% 71 Missing :warning:
src/Console/DumpDatabaseCommand.php 0.00% 19 Missing :warning:
src/IlluminateRegistry.php 70.00% 12 Missing :warning:
src/EntityManagerFactory.php 87.91% 11 Missing :warning:
src/Notifications/Notification.php 0.00% 10 Missing :warning:
src/Testing/Factory.php 47.36% 10 Missing :warning:
src/Configuration/MetaData/Fluent.php 0.00% 7 Missing :warning:
src/Testing/Concerns/InteractsWithEntities.php 53.33% 7 Missing :warning:
src/Queue/FailedJobsServiceProvider.php 0.00% 6 Missing :warning:
src/Validation/PresenceVerifierProvider.php 0.00% 6 Missing :warning:
... and 32 more

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## 3.0.x #582 +/- ## ======================================== Coverage ? 64.04% Complexity ? 531 ======================================== Files ? 76 Lines ? 1677 Branches ? 0 ======================================== Hits ? 1074 Misses ? 603 Partials ? 0 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

TomHAnderson commented 1 month ago

Merging this without static analysis. Will be addressed in future PR