-
-
Notifications
You must be signed in to change notification settings - Fork 155
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
Move some namespaces on new Metadata #774
Move some namespaces on new Metadata #774
Conversation
loic425
commented
Nov 6, 2023
Q | A |
---|---|
Bug fix? | no |
New feature? | no |
BC breaks? | no |
Deprecations? | no |
Related tickets | |
License | MIT |
97aa866
to
9ba4eb4
Compare
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.
LGFM, but I don't know how to validate that everything is good, since it's a big search/replace.
Yes, these classes have been introduced in 1.11 so, they don't need any bc-layer. |
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've browsed around 40%, will finish later this day
src/Component/Symfony/Routing/Factory/OperationRouteFactoryInterface.php
Show resolved
Hide resolved
src/Component/spec/Symfony/Routing/Factory/CollectionOperationRoutePathFactorySpec.php
Outdated
Show resolved
Hide resolved
src/Component/spec/Symfony/Routing/Factory/CreateOperationRoutePathFactorySpec.php
Outdated
Show resolved
Hide resolved
src/Component/spec/Symfony/Routing/Factory/DeleteOperationRoutePathFactorySpec.php
Outdated
Show resolved
Hide resolved
src/Component/spec/Symfony/Routing/Factory/DeleteOperationRoutePathFactorySpec.php
Outdated
Show resolved
Hide resolved
src/Component/spec/Symfony/Routing/Factory/UpdateOperationRoutePathFactorySpec.php
Outdated
Show resolved
Hide resolved
src/Component/spec/Symfony/Routing/Factory/UpdateOperationRoutePathFactorySpec.php
Outdated
Show resolved
Hide resolved
741361a
to
818d8f3
Compare
c769b5d
to
91e7768
Compare
Thank you, @loic425! |