-
Notifications
You must be signed in to change notification settings - Fork 58
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
11.0 release #107
11.0 release #107
Conversation
…uch easier to expand on additional features and testing.
…velopers to setup their query logs.
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.
@kirkbushell this is very nice PR, with attention to detail and the new implementations are really good! The code comments helped me a lot while going through it.
I left some comments about typos, some improvements in terms of simplification and readability, and a couple of questions, but everything seems well-thought and I couldn't find any significant issues 🙂
I am really looking forward for this to go out!
P.S. I finally am aware of all the capabilities of this amazing package!
Moving to release branch. |
Currently work in progress, but this PR encapsulates the complete rework of the Eloquence solution, using a much nicer and workable API with modern approaches.
The main objective of version 11 is to prepare for Laravel 11 (in general), whilst introducing a nicer API and easier way of working with the main camelcasing trait, and providing better support for its various data aggregation features.
Due to the amount of changes, backwards compatibility is not possible.
Upgrade guide to come.