-
Notifications
You must be signed in to change notification settings - Fork 157
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
Update json-schema-ref-parser dependency #30
Comments
I'm hoping to get to this this weekend. I've been waiting to finish this PR first. As of this commit, it's mostly done. Just need to update the documentation, and find out why some tests are failing in a few browsers |
👍 |
+1.
just to give you an idea of the test… var jsonSchemaRefParser = require('json-schema-ref-parser');
var _ = require('lodash');
var async = require('async');
var results = [];
async.timesSeries(1000, function (n, next) {
'use strict';
jsonSchemaRefParser.bundle(__dirname + '/api/index.yaml')
.then(function (api) {
results.push(JSON.stringify(api.definitions));
next();
});
}, function (err) {
'use strict';
var length = results.length;
console.log('length === 1000? ' + (length === 1000));
var uniqArray = _.uniq(results);
var uniqLength = uniqArray.length;
console.log('unique length === 1? ' + (uniqLength === 1));
console.log(uniqArray);
}); |
@BigstickCarpet it blocks me. Could I help you with this? |
I just stabilized JSON Schema $Ref Parser v3.0.0. So tomorrow I'll start getting Swagger Parser updated |
Awesome! Thanks for quick reply :) |
Swagger Parser v4.0 beta is now available. |
Thanks, now it's fine. Closing this issue. |
Currently swagger-parser uses [email protected].
Could you please update it to use the latest version?
The text was updated successfully, but these errors were encountered: