SncRedisBundle icon indicating copy to clipboard operation
SncRedisBundle copied to clipboard

[Dependency] Added conflict with predis >= 3.0

Open popy-dev opened this issue 6 months ago • 4 comments

As of today, redis-bundle doesn't implements the latest predis interface. The suggested update will prevent installing/updating this version.

This would fix issue #735 until #736 gets finalized/merged.

popy-dev avatar May 26 '25 08:05 popy-dev

It won't really fix anything because Composer will just use an older version of the bundle then that has no conflict. This should have been there from the start ideally, but still good to merge IMO for clarity.

Seldaek avatar May 28 '25 07:05 Seldaek

It won't really fix anything because Composer will just use an older version of the bundle then that has no conflict. This should have been there from the start ideally, but still good to merge IMO for clarity.

Nope, composer did not use an older version as it has no idea it has to do it. I had the issue on a project :)

popy-dev avatar May 28 '25 09:05 popy-dev

No what I mean is that if we merge/release this PR, a composer update would still then possibly upgrade to predis/predis 3.x if your requirements allow it, but it would keep this bundle at version 2.9.0 and not upgrade it to the 2.9.1 that has a conflict against predis 3.

Seldaek avatar May 28 '25 09:05 Seldaek

not needed thanks to https://github.com/snc/SncRedisBundle/pull/736

ostrolucky avatar May 28 '25 11:05 ostrolucky