cas-bundle icon indicating copy to clipboard operation
cas-bundle copied to clipboard

Update composer.json for SF7

Open StudioAH opened this issue 1 year ago • 8 comments

dependencies updates for symfony 7

This PR

  • [x]
  • [ ]
  • [ ]

Follows #. Related to #97. Fixes #.

StudioAH avatar Feb 20 '24 07:02 StudioAH

Hi, previous version was already compatible with Symfony 6 so why not following up with 7 ?

StudioAH avatar Feb 20 '24 08:02 StudioAH

Hello,

Why did you close the PR ?

FYI, I received a request for making it compatible with SF7 yesterday, I gave more information in the issue itself at https://github.com/ecphp/cas-bundle/issues/97

drupol avatar Feb 20 '24 09:02 drupol

Hello, No testing was done for both SF 6 & 7 and the code had to be changed. Therefore I wanted to make sure this would work. But if you want to work on it be my guest because I am struggling to test it locally atm.

StudioAH avatar Feb 20 '24 09:02 StudioAH

I'm confused.

If you open a PR against this project, I guess that's because you want some stuff to be updated in it, right?

drupol avatar Feb 20 '24 09:02 drupol

Yes but your comment was right, my code was not considering the bundle to work for both symfony 6 and 7 at the same time. It needed some changes ^6.0 | ^7.0 for exemple

StudioAH avatar Feb 20 '24 09:02 StudioAH

OK OK !

Yeah if you could reopen your PR and update to ^6.0 || ^7.0 (mind the 2 |), it could be nice, so you could tell me if it works, and if it is successful, I can cut a release and also close the aforementionned issue!

drupol avatar Feb 20 '24 09:02 drupol

Nice! Thank you very much.

@Punt13140 Do you mind testing that branch and see if it works on your side? In case of a yes, I'll cut a new release immediately.

@StudioAH No worries about the broken tests, I'll take care of it myself.

drupol avatar Feb 20 '24 10:02 drupol

I quickly tried to install the bundle in a new Symfony 7 project, and it doesn't work out of the box.

I have dependency problems with

"psr/http-message": "^1.0",

Trying to change

"psr/http-message": "^1.0|^2.0",

We have another dependency problem with

loophp/psr-http-message-bridge-bundle

which depends on "symfony/psr-http-message-bridge": "^1 || ^2"

However, it seems that the latter now follows Symfony tags since 6.4...

Import the bridge into the Symfony monorepo and synchronize releases https://github.com/symfony/psr-http-message-bridge/blob/v6.4.3/CHANGELOG.md

smbpunt avatar Feb 20 '24 16:02 smbpunt