-
-
Notifications
You must be signed in to change notification settings - Fork 233
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
Addition of dereferencedCache has broken deeply nested external dependencies #226
Comments
In the example above I think this is due to it matching the |
FYI @paztis |
philsturgeon
pushed a commit
to mummybot/json-schema-ref-parser
that referenced
this issue
May 4, 2021
mummybot
added a commit
to mummybot/json-schema-ref-parser
that referenced
this issue
Jun 8, 2021
…-parser into bug/APIDevTools#226/external-from-external
mummybot
added a commit
to mummybot/json-schema-ref-parser
that referenced
this issue
Jun 8, 2021
…om:mummybot/json-schema-ref-parser into bug/APIDevTools#226/external-from-external
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
The addition of the
dereferencedCache
in PR #195 has broken deeply nested external dependencies if a local dependency exists in the root schema.Given the following schema:
Expected
Actual
If I comment out https://github.com/APIDevTools/json-schema-ref-parser/blob/master/lib/dereference.js#L177 then it all works (by not adding to cache and therefore not skipping).
The text was updated successfully, but these errors were encountered: