Skip to content

Conversation

@cHolzberger
Copy link
Contributor

MR changes discussed in #6.
Dereferencing is explicit now either throuh swagger.deref or swagger.derefSync.

Still one problem left. $ref: "ext.yml#/path" is not working, still looking for the reason.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.5%) to 99.469% when pulling 51bdc9c on cHolzberger:local-ref-take2 into 6e83fec on carlansley:master.

@carlansley
Copy link
Owner

Sorry it's been a while since I looked at this, are you done with this?

@linusnorton
Copy link

@carlansley Is there any way we can look at getting this intergated?

@carlansley
Copy link
Owner

@linusnorton if the PR can be updated to resolve the remaining issue(s) happy to merge it. We have an internal replacement for swagger2 that supports openapi 3 and also external refs which we'll be making public in the next couple of weeks.

@linusnorton
Copy link

In that case I'm happy to wait, thanks for the update.

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.

4 participants