alice icon indicating copy to clipboard operation
alice copied to clipboard

MEzzio/Laminas bridge

Open func0der opened this issue 3 years ago • 2 comments

This is a bridge for nelmio/alice in Laminas.

Maybe the bridges should lay outside of this repo completely? Generally this should help laminas developers a lot.

func0der avatar Mar 29 '21 11:03 func0der

How is the status about this, @theofidry ?

func0der avatar Aug 10 '21 11:08 func0der

@func0der sorry for the late reply. The code looks good to me. I'm however slightly uncomfortable to merge this without any test whatsoever, would it be possible to have an equivalent to the Symfony LoaderIntegrationTest one? It's not a requirement, but without it as I'm not a Laminas user I won't be able to guarantee if anything breaks for it.

Maybe the bridges should lay outside of this repo completely? Generally this should help laminas developers a lot.

Yes and no: in the end it's yet another repository to manage and it's quite annoying to realise you are breaking something for a peculiar bridge and need to come back to fix something in the library for it. In this case, the framework bridges are also very thin layers: they simply register services to the framework DI. Unlike HautelookAliceBundle for example which relies a lot more on the Symfony API to provide more Symfony specific features.

theofidry avatar Sep 26 '21 17:09 theofidry

Revisiting this, I think there is just not enough interest for this bridge hence I'll close it down for now. If there is someone interested in it this can be revisited.

theofidry avatar Sep 18 '23 20:09 theofidry

@lkck24 FYI

func0der avatar Sep 19 '23 11:09 func0der