Skip to content

Add Symfony 5 support#627

Closed
k0d3r1s wants to merge 1 commit into
googleads:masterfrom
k0d3r1s:master
Closed

Add Symfony 5 support#627
k0d3r1s wants to merge 1 commit into
googleads:masterfrom
k0d3r1s:master

Conversation

@k0d3r1s

@k0d3r1s k0d3r1s commented Nov 23, 2019

Copy link
Copy Markdown

Add Symfony 5 support

#SymfonyHackday

@fiboknacky fiboknacky requested a review from thangduo November 24, 2019 06:57
@fiboknacky

Copy link
Copy Markdown
Member

@k0d3r1s

I found PHPUnit fails with Symfony 5.0.
Could you also address the failed tests first and attach the results here?

@fiboknacky fiboknacky requested review from fiboknacky and removed request for thangduo November 25, 2019 05:33

@fiboknacky fiboknacky left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

PHPUnit tests fail. Please fix them first.

@k0d3r1s

k0d3r1s commented Dec 4, 2019

Copy link
Copy Markdown
Author

will take a look when have free time

@fiboknacky fiboknacky mentioned this pull request Jan 9, 2020
@fiboknacky

Copy link
Copy Markdown
Member

will take a look when have free time
@k0d3r1s
Any updates?

@k0d3r1s

k0d3r1s commented Jan 9, 2020

Copy link
Copy Markdown
Author

unfortunetly not. there is something with the way this lib uses symfony library but I haven't got to actually debugging / fixing this issue

@fiboknacky

Copy link
Copy Markdown
Member

This is merged in #638.

@fiboknacky fiboknacky closed this Jan 17, 2020
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