Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix: ref already seen schemas in deref walk #438

Merged
merged 1 commit into from
Aug 22, 2024
Merged

fix: ref already seen schemas in deref walk #438

merged 1 commit into from
Aug 22, 2024

Conversation

nrwiersma
Copy link
Member

This references previously dereferenced schemas when parsing.

There is an edge case, where in multiple parse calls, a schema has been dereferenced twice. If this ends up in the same schema, the second "copy" needs to be re-referenced.

Fixes #437

@anekdoti
Copy link

Thank you, @nrwiersma! This does the trick :-)

@nrwiersma nrwiersma merged commit c2ac60e into main Aug 22, 2024
3 checks passed
@nrwiersma nrwiersma deleted the deref-schema branch August 22, 2024 15:42
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

The ParseFiles function is generating duplicate schema names when processing multiple source files
2 participants