Skip to content

Add support for Symfony/YAML 4.4 #61

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

Merged
merged 2 commits into from
May 27, 2020
Merged

Add support for Symfony/YAML 4.4 #61

merged 2 commits into from
May 27, 2020

Conversation

recchia
Copy link
Contributor

@recchia recchia commented Mar 10, 2020

Add support for Symfony/YAML 4.4, this is a transition version to 5. In 4.4 you can fix/remove deprecated code before migrate to version 5

@aberlinski
Copy link

Hello @cebe

Could you please tell me when this pull request is going to be merged? We want to use cebe/php-openapi in our project, but lack of symfony/Yaml 4.4 support blocks us. I would be appreciate to merge it as quick as it possible :)

@cebe
Copy link
Owner

cebe commented May 27, 2020

I was explicitly excluding this version because of bugs in the yaml implementation, see #60

If it does not break my tests I can add it back, need to check it.

@cebe
Copy link
Owner

cebe commented May 27, 2020

updated composer.json, lets see if tests are green. Tests are running on 4.3 anyway so that should work.

@cebe
Copy link
Owner

cebe commented May 27, 2020

https://travis-ci.org/github/cebe/php-openapi/builds/691798677

tests are fine, seems phpstan is broken...

@cebe cebe merged commit 2cf994d into cebe:master May 27, 2020
@cebe
Copy link
Owner

cebe commented May 27, 2020

Merged, Thank you!

@cebe cebe added this to the 1.4.2 milestone May 27, 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