Problem/Motivation
As requested by @catch in #3327769-4: Upgrade Symfony dependencies to 6.2.2 now in a separate issue: Update doctrine/annotations-dependency.
Steps to reproduce
Proposed resolution
~/.config/composer/vendor/bin/composer-lock-diff --no-links
+----------------------+--------+--------+
| Production Changes | From | To |
+----------------------+--------+--------+
| doctrine/annotations | 1.13.3 | 1.14.2 |
+----------------------+--------+--------+
Remaining tasks
User interface changes
API changes
Data model changes
Release notes snippet
| Comment | File | Size | Author |
|---|---|---|---|
| #15 | 3327787.10.1.x-15.patch | 4.84 KB | spokje |
Comments
Comment #2
spokjeComment #3
spokjeComment #4
andypostLooking at release notes https://github.com/doctrine/annotations/releases
Minor release 1.14.0 added forward compatibility with lexer v2 https://github.com/doctrine/annotations/pull/460
Comment #5
spokjeComment #6
xjmLet's also update the constraint for this. (For the past several minors, we've adopted a best practice of setting the minor version constraints to the highest working version with each new core minor.) Thanks!
Comment #7
spokjeFine by me :)
Comment #8
xjmThis issue also reminded me that I want us to work on #3252386: Use PHP attributes instead of doctrine annotations.
Comment #10
xjmAh, I bet the hashes need to be regenerated due to the Symfony patch release commit. This will also be true with the Doctrine issue for whichever of these two lands first.
Comment #11
spokjeNew version out, let's make this a 10.1.x patch first to see what happens.
Comment #12
spokjeUnsure where the "sudden" need for a hash change came from, but it's (again) unneeded now.
Comment #13
andypostnice to see that)
Comment #14
catchNeeds a re-roll. #13 confirms why #3252386: Use PHP attributes instead of doctrine annotations needs to be critical - we need to phase out annotations before Doctrine completely drops it.
Comment #15
spokjeRerolled
Comment #16
spokjeComment #17
catchCommitted bcbd7ba and pushed to 10.1.x. Thanks!
Comment #19
catchComment #20
catchComment #22
xjmThis change is covered by the standard minor release note of: