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

Pin openapi3_parser #121

Merged
merged 1 commit into from
Oct 9, 2019
Merged

Pin openapi3_parser #121

merged 1 commit into from
Oct 9, 2019

Conversation

36degrees
Copy link
Contributor

The 0.6.0 release is causing builds to fail. Pinning it to 0.5.2 appears to solve this.

The maintainer of openapi3_parser has been made aware, and hopes to have a new version out soon.

The 0.6.0 release is causing builds to fail. Pinning it to 0.5.2 appears to solve this.

The maintainer of openapi3_parser has been made aware, and hopes to have a new version out soon.
@36degrees
Copy link
Contributor Author

Mostly raising this to check that it does fix the failing tests currently on master.

If a new release of openapi3_parser is released soon, we may not need to merge this.

@kevindew
Copy link
Member

kevindew commented Oct 9, 2019

@36degrees I fixed the underlying bug in openapi_parser with 0.6.1 however I discovered (much to my surprise) that this gem also relies on a method that was removed, so I'd suggest you merge this in.

I'm planning to update the code in this gem for compatibility without this method and will open a fresh PR for that (after thoroughly rasping myself for not realising the significance of a BC break) and can update the gem dependency then too.

Apologies for the inconveniences.

@36degrees 36degrees marked this pull request as ready for review October 9, 2019 08:55
@36degrees 36degrees merged commit 95e0cae into master Oct 9, 2019
@36degrees 36degrees deleted the pin-openapi-parser branch October 9, 2019 12:13
@kevindew kevindew mentioned this pull request Oct 14, 2019
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.

2 participants