reference remote files about swagger-merger HOT 4 CLOSED

Good idea 👍 , I tried to implement this idea in v1.3.0

Welcome feedback.

from swagger-merger.

Hi @WindomZ ,
thank you for such a fast fix, again 👍
Sadly it's not working for me this time.

doesn't matter where i try to reference, i always get this error for the following merge using the newest 1.3.0:

$ node_modules/swagger-merger/bin/swagger-merger.js -i index.yaml -o test.yaml --debug

I tried it on my own file as well as this index.yaml from the examples without the local references (paths, definitions)

SyntaxError: Unexpected token U in JSON at position 271
    at Object.parse (native)
    at mergeJSON (..\node_modules\swagger-merger\lib\merge_json.js:26:15)
    at mergerYAML (..\node_modules\swagger-merger\lib\merger_yaml.js:20:13)
    at merge (..\node_modules\swagger-merger\lib\merger.js:52:14)
    at merger (..\node_modules\swagger-merger\lib\merger.js:34:3)
    at next (native)
    at onFulfilled (..\node_modules\co\index.js:65:19)
    at ..\node_modules\co\index.js:54:5
    at co (..\node_modules\co\index.js:50:10)
    at createPromise (..\node_modules\co\index.js:30:15)

might be something like this
But i didn't understand completely what is happening in lines 26 and 27 of merge_json.js

from swagger-merger.

I tested it today on a fresh new linux installation (other was with windows in gitbash where i often run into some problems).
It worked fine :)

from swagger-merger.

I have found some problems in Windows

from swagger-merger.