Drop `laminas/laminas-zendframework-bridge` and `zendframework/*` compatibility

mezzio / mezzio-fastroute

FastRoute integration for Mezzio

https://docs.mezzio.dev/mezzio/features/router/fast-route/

BSD 3-Clause "New" or "Revised" License

16 stars 11 forks source link

Drop `laminas/laminas-zendframework-bridge` and `zendframework/*` compatibility #15

Closed PowerKiKi closed 2 years ago

PowerKiKi commented 2 years ago

Q	A
Documentation	no
Bugfix	yes
BC Break	no
New Feature	no
RFC	no
QA	no

Description

Increase performance by removing a compatibility layer while not introducing breaking changes.

This follow the process described in details in:

https://github.com/laminas/technical-steering-committee/blob/main/meetings/minutes/2021-08-02-TSC-Minutes.md#remove-laminaslaminas-zendframework-bridge-dependency-from-our-packages

PowerKiKi commented 2 years ago

@Ocramius, since you merged many of similar PR, would you mind having a look at this one too, when you have time for it, please ?

This is the last one that I really care about and that didn't get any feedback yet. However the are a few more over there if somebody want to go over all of them at once: https://github.com/pulls?q=is%3Aopen+is%3Apr+author%3APowerKiKi+archived%3Afalse++bridge

Ocramius commented 2 years ago

@PowerKiKi I mostly operated from mail notifications: because all the PRs had the same title, GMail got really confused about it, and started grouping some together 🤷