search

amphp / injector

A recursive dependency injector used to bootstrap and wire together S.O.L.I.D., object-oriented PHP applications.
MIT License
31 stars 7 forks source link

Conflict with auryn #8

Closed szepeviktor closed 3 years ago

kelunik commented 3 years ago

As it's a new namespace, I don't see a reason to add such a conflict rule. Where do you see a conflict?

szepeviktor commented 3 years ago

When a user forgets to remove auryn while upgrading.

kelunik commented 3 years ago

Forgetting to remove Auryn doesn't cause any issues except for an additionally installed library, but adding the conflict rule might cause issues if package A depends on rdlowrey/auryn and package B depends on amphp/injector, while this is perfectly fine and working without issues.

szepeviktor commented 3 years ago

Okay.