-
Notifications
You must be signed in to change notification settings - Fork 33
Version 2.0.x support of Laravel Doctrine ORM ^3.1 and Laravel 12 #73
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
Conversation
R3VoLuT1OneR
commented
Apr 26, 2025
- Support of Laravel 12
- Support of Laravel Doctrine ORM ^3.1
- Migrated to attributes instead of annotations
- PHPUnit setup workbench for laravel imulation
- GitHub actions setup php tests on github
- PHPCS, PHPStan upgrade coding standards
- Updated README.md
- Support of Laravel 12 - Support of Laravel Doctrine ORM ^3.1 - Migrated to attributes instead of annotations - PHPUnit setup workbench for laravel imulation - GitHub actions setup php tests on github - PHPCS, PHPStan upgrade coding standards - Updated README.md
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## 2.0.x #73 +/- ##
========================================
Coverage ? 98.73%
Complexity ? 138
========================================
Files ? 23
Lines ? 316
Branches ? 0
========================================
Hits ? 312
Misses ? 4
Partials ? 0 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
@TomHAnderson I resolved all the conversations I fixed and considered small or should I keep them open so you will resolve them? |
I'll go through and resolve them. I'm out today but will look at this again tonight. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Really good work on this. Looking for your feedback on these final two notes.
1e5581c
to
6ee3ab6
Compare
I'm ready to approve this. Are you ready for a 2.0.0 release when I do? |
Great, I think we can release it yes. |