-
Notifications
You must be signed in to change notification settings - Fork 41
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
Support Symfony 7 #318
Support Symfony 7 #318
Conversation
1b6f9dc
to
eb7dd61
Compare
eb7dd61
to
573bac3
Compare
@peter-gribanov Please review |
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.
Thank you.
I am happy with this.
Maybe @peter-gribanov wants to have a look too as he is the active maintainer.
Guys, we need this PR, cause any team can't use symfony 7.X |
just go with this fork: "repositories": [
{
"type": "vcs",
"url": "https://github.com/Niiph/Doctrine-Specification"
}
] it's not ideal solution but should give you ability to go with Symfony 7 |
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.
I'm happy with this.
I can't merge atm. On my phone and CI is not 100% green
Thank you |
Since
symfony/property-access
7.0 requires php8.2 and php8.2 had some changes in datetime format: https://www.php.net/manual/en/datetime.formats.php (end of document) I had to make some changes@peter-gribanov