Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Symfony 5 support#31

Merged
kbond merged 3 commits into
zenstruck:masterfrom
napestershine:patch-1
Aug 27, 2020
Merged

Symfony 5 support#31
kbond merged 3 commits into
zenstruck:masterfrom
napestershine:patch-1

Conversation

@napestershine
Copy link
Copy Markdown
Contributor

No description provided.

Copy link
Copy Markdown

@weaverryan weaverryan left a comment

Choose a reason for hiding this comment

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

Happy to see the PR - we will need it very soon :)

Comment thread composer.json Outdated
Comment thread composer.json Outdated
@kbond
Copy link
Copy Markdown
Member

kbond commented Aug 27, 2020

Thanks @napestershine, don't worry about the failing travis build - it's a known issue I need to look into.

@weaverryan should we keep Symfony 3.4 support?

@kbond
Copy link
Copy Markdown
Member

kbond commented Aug 27, 2020

Never mind, I see Ryan suggested only supporting 4.4+. I'm good with that.

@napestershine, can you change the symfony package constraints from ^4.0|^5.0 to ^4.4|^5.0?

@napestershine
Copy link
Copy Markdown
Contributor Author

@kbond I have updated it.

@kbond kbond merged commit f7c1404 into zenstruck:master Aug 27, 2020
@kbond
Copy link
Copy Markdown
Member

kbond commented Aug 27, 2020

Thanks, tagged v3.4.0.

@napestershine napestershine deleted the patch-1 branch August 27, 2020 15:37
@napestershine
Copy link
Copy Markdown
Contributor Author

@kbond I am getting exact same errors on my installation shown by Travis.

@kbond
Copy link
Copy Markdown
Member

kbond commented Sep 3, 2020

I see a lot of different travis errors - can you link to the specific one?

@kbond
Copy link
Copy Markdown
Member

kbond commented Sep 10, 2020

@napestershine, I refactored and got the CI tests working again - can you try v3.5.0?

@napestershine
Copy link
Copy Markdown
Contributor Author

Yes sure. I will try it and update you. Thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants