Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow sensio/framework-extra-bundle version 6 #2551

Merged
merged 2 commits into from Apr 27, 2021

Conversation

bobvandevijver
Copy link
Contributor

I'm not sure if PSR-7 is actually used by this bundle, but that is the only major change between v5 en v6 of sensio/framework-extra-bundle (see sensiolabs/SensioFrameworkExtraBundle#710 / https://github.com/sensiolabs/SensioFrameworkExtraBundle/blob/master/CHANGELOG.md).

It seems to work fine for me when forcing composer to install v6.1.2.

@bobdenotter
Copy link
Member

Hi, @bobvandevijver 👋

I personally don't like PSR-7, and I'm confident Symfony doesn't use it either.

Shouldn't we just bump it to ^6.1? I'm not a fan of having || dependencies in Core. (long story short, it adds overhead in resolution, and it makes providing bolt/project with a lock file much harder and more fragile.

@bobvandevijver
Copy link
Contributor Author

Hi 👋

Bumping it to 6.1 is fine by me 👍🏻

@bobdenotter
Copy link
Member

bobdenotter commented Apr 27, 2021

Cool! If tests are passing, I'll merge it in.

Thanks!

@bobvandevijver
Copy link
Contributor Author

You might want to squash it before merge (if that isn't enabled by default), as two commits for this change might be a bit too much 😅

@bobdenotter bobdenotter merged commit fda3193 into bolt:4.1 Apr 27, 2021
@bobdenotter
Copy link
Member

Done, thanks again! 👍

@bobdenotter bobdenotter changed the title Allow sensio/framework-extra-bundle version 6 Allow sensio/framework-extra-bundle version 6 May 11, 2021
napley pushed a commit to novaway/core that referenced this pull request May 19, 2021
* Allow sensio/framework-extra-bundle version 6

* Bump sensio/framework-extra-bundle to 6.1
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.

None yet

2 participants