search

apiaryio / fury-adapter-swagger

Swagger 2.0 parser adapter for Fury.js
MIT License
11 stars 12 forks source link

Update swagger-parser to version 4 #158

Closed realityking closed 5 years ago

realityking commented 6 years ago

Resolves #157

Unfortunately test fails due to BigstickCarpet/swagger-parser#76 but I wanted to document the progress 😆

In case swagger-parser doesn't add the option, swagger-zoo would have to be changed. Specifically this file https://github.com/apiaryio/swagger-zoo/blob/master/fixtures/features/api-elements/path-reference-typo.json and its source map.

realityking commented 6 years ago

Now that BigstickCarpet/json-schema-ref-parser#75 is merged, this should soon be unblocked. Just have to wait for things to be released and to tickle up.

pksunkara commented 6 years ago

The json-schema-ref-parser v5 needs to be propagated into swagger-parser and then a new version needs to be released

realityking commented 6 years ago

It's already in master of swagger-parser. There's some build issue according to swagger-parser#85.

realityking commented 6 years ago

Unfortunately my fix was incomplete. Now waiting for https://github.com/BigstickCarpet/json-schema-ref-parser/pull/80 to be merged.