Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Circular references in separate files results in stack overflow error #856

Closed
reuvenharrison opened this issue Nov 24, 2023 · 3 comments · Fixed by #974
Closed

Circular references in separate files results in stack overflow error #856

reuvenharrison opened this issue Nov 24, 2023 · 3 comments · Fixed by #974

Comments

@reuvenharrison
Copy link
Contributor

See Tufin/oasdiff#442
fenollp added a commit to fenollp/kin-openapi that referenced this issue Nov 24, 2023
@fenollp
reproduce getkin#856
5dccbb8 
Signed-off-by: Pierre Fenoll <[email protected]>
@fenollp fenollp linked a pull request Nov 24, 2023 that will close this issue
reproduce #856 #857
Closed
fenollp added a commit to fenollp/kin-openapi that referenced this issue Nov 26, 2023
@fenollp
WIP: reproduce getkin#856
eda2c93 
Signed-off-by: Pierre Fenoll <[email protected]>
fenollp added a commit to fenollp/kin-openapi that referenced this issue Nov 26, 2023
@fenollp
WIP: reproduce getkin#856
7806af9 
Signed-off-by: Pierre Fenoll <[email protected]>
fenollp added a commit to fenollp/kin-openapi that referenced this issue Nov 26, 2023
@fenollp
WIP: reproduce getkin#856
c035c42 
Signed-off-by: Pierre Fenoll <[email protected]>
fenollp added a commit to fenollp/kin-openapi that referenced this issue Nov 26, 2023
@fenollp
WIP: reproduce getkin#856
df23341 
Signed-off-by: Pierre Fenoll <[email protected]>
@vdlp-mw
Copy link

vdlp-mw commented Jan 23, 2024

Are there any updates on this? Anything I may be able to do to get this issue resolved?

@AnatolyRugalev
Copy link
Contributor

Can you check if #970 solves the issue for you?

@reuvenharrison
Copy link
Contributor Author

Hi @AnatolyRugalev
I checked your fix.
Unfortunately it doesn't seem to resolve this issue.
Please see more details here: #973

@AnatolyRugalev AnatolyRugalev mentioned this issue Jul 1, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
3 participants
@AnatolyRugalev @reuvenharrison @vdlp-mw