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.

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

test(ls): demonstrate dereference issue with AsyncAPI #3110

Closed
wants to merge 5 commits into from

Conversation

frantuma
Copy link
Member

@frantuma frantuma commented Sep 3, 2023

@char0n

This (failing) PR demonstrates an issue with dereferencing AsyncAPI.

can you possibly help?

@char0n
Copy link
Member

char0n commented Sep 3, 2023

@frantuma will try to look into in tommorow.

@char0n
Copy link
Member

char0n commented Sep 20, 2023

The issue is that our visitor mechanism was not fully immutable and was failing on frozen ApiDOM.

char0n added a commit that referenced this pull request Sep 21, 2023
char0n added a commit that referenced this pull request Sep 29, 2023
Part of this PR addresses alignment with dereference strategies
and new immutable traversal.

Refs #3110
@char0n
Copy link
Member

char0n commented Sep 29, 2023

Closing as the test is passing when run locally.

@char0n char0n closed this Sep 29, 2023
@char0n char0n self-assigned this Sep 29, 2023
@char0n char0n deleted the issue-deref branch October 17, 2023 11:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants