Skip to content

Conversation

alexandre-daubois
Copy link
Member

@alexandre-daubois alexandre-daubois commented Apr 30, 2024

Q A
Branch? 5.4
Bug fix? no
New feature? no
Deprecations? no
Issues -
License MIT

Last round (before new deprecations 🙂). All deprecations listed here should be gone.

@carsonbot carsonbot added this to the 5.4 milestone Apr 30, 2024
@alexandre-daubois alexandre-daubois force-pushed the pu-double-deprecs branch 4 times, most recently from 62141d7 to 05be1c1 Compare May 2, 2024 07:11
@alexandre-daubois alexandre-daubois force-pushed the pu-double-deprecs branch 2 times, most recently from 25bc856 to 7e9804d Compare May 3, 2024 08:11
@alexandre-daubois alexandre-daubois force-pushed the pu-double-deprecs branch 3 times, most recently from 78d0ccf to 1072546 Compare May 3, 2024 08:41
Copy link
Member

@derrabus derrabus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry for reviewing in small chunks. This is hopefully the last batch of nit-picks. 😓

@alexandre-daubois
Copy link
Member Author

No worries, these comments bring elegant code so that's nice to update 😄

Copy link
Member

@derrabus derrabus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fabbot's comment about the closure syntax is valid, that was my bad. 😓

Apart from that: LGTM.

@xabbuh
Copy link
Member

xabbuh commented May 15, 2024

Thank you @alexandre-daubois.

@xabbuh xabbuh merged commit ac30c7e into symfony:5.4 May 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants