Sylius / PayPalPlugin

Official integration with PayPal Commerce Platform
37 stars 56 forks source link

[Bug] Wrong dependency #232

Open Nek- opened 2 years ago

Nek- commented 2 years ago

SyliusPayPalPlugin version affected: 1.4.x

Description

I may be wrong but I think using a dependency to mock the container in the standard require section of the composer.json file sounds wrong:

https://github.com/Sylius/PayPalPlugin/blob/b32c831b6cb0dea933e87ec130b2006c63567e4b/composer.json#L7-L14

I guess it's not super-super safe to have it in production as well? (didn't check anything, but since it's testing stuff...)

Possible Solution

Moving it to require-dev section.