Skip to content

Conversation

popy-dev
Copy link

@popy-dev popy-dev commented May 26, 2025

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 popy-dev changed the title [Dependency] [Dependency] Added conflict with predis >= 3.0 May 26, 2025
@Seldaek
Copy link
Collaborator

Seldaek commented May 28, 2025

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.

@popy-dev
Copy link
Author

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 :)

@Seldaek
Copy link
Collaborator

Seldaek commented May 28, 2025

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.

@ostrolucky
Copy link
Collaborator

not needed thanks to #736

@ostrolucky ostrolucky closed this Jun 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants