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

composer: allow Symfony 5 #1242

Merged
merged 1 commit into from
Nov 27, 2019
Merged

composer: allow Symfony 5 #1242

merged 1 commit into from
Nov 27, 2019

Conversation

TomasVotruba
Copy link
Contributor

We wanted to ugprade to Symfony 5 released yesterday, but this bundle is blocking us :/

@michellesanver
Copy link
Contributor

Hi, thanks a lot for your contribution Tomas! :)

@michellesanver michellesanver merged commit 65e6992 into liip:master Nov 27, 2019
@TomasVotruba TomasVotruba deleted the patch-1 branch November 27, 2019 15:53
@TomasVotruba
Copy link
Contributor Author

Hey Michelle 🌈 🦄

Long time no see, have a beautiful night 🌟

@dbu
Copy link
Member

dbu commented Nov 27, 2019

erm, the build is failing: PHP Fatal error: Uncaught Error: Class 'Symfony\Bundle\FrameworkBundle\Tests\TestCase' not found in /home/travis/build/liip/LiipImagineBundle/Tests/Exception/Imagine/Filter/PostProcessor/InvalidOptionExceptionTest.php:20

i fear there is some change with symfony 5 that needs some hackery to support both 4 and 5. i did some nice horrible things for FriendsOfSymfony/FOSHttpCacheBundle#533 which might serve as inspiration.

i'd recommend to add a flex build that forces symfony 5 - currently it seems like some components got installed with version 4, likely because version constraints in dependencies.

it is problematic if composer.json allows to install with symfony 5 when it might not actually works.

@michellesanver
Copy link
Contributor

Oh shit, somehow missed that, got too used to gitlab 😐 I’ll revert this when I’m back at my computer in about an hour.

@michellesanver
Copy link
Contributor

Ok reverted I’ll look after dinner how to get this to run on SF 5.

@dbu
Copy link
Member

dbu commented Nov 27, 2019

thanks. i might be able to help a bit tomorrow. if needed, ping me.

@michellesanver
Copy link
Contributor

Scratch the after dinner part. I’ll look tomorrow during work hours. My son needs me more than imagine does. If it’s urgent for anyone in the meanwhile feel free to fix it and ping me for a review.

@michellesanver
Copy link
Contributor

Actually, as the original contributor is this something you would be willing to contribute @TomasVotruba and yeah, long time no see! 💚🦄 🌈

@TomasVotruba
Copy link
Contributor Author

I don't have time now, preparing first public Symfony training tomorrow

@michellesanver
Copy link
Contributor

No worries! I’ll look tomorrow, and good luck with your training 😁👍🏻

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