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

fix: on_attach called multiple times with separate_diagnostic_server #220

Merged
merged 3 commits into from
Jan 14, 2024

Conversation

pmizio
Copy link
Owner

@pmizio pmizio commented Jan 13, 2024

Fixes: #217 #219
Reverts: #213 #218

Hey @b0o @chaozwn, in here I reverted all changes related to on_attach problems and introduced I think cleaner and proper/less hacky fix for it.
Can you verify it? I will be really grateful!

@pmizio pmizio self-assigned this Jan 13, 2024
@chaozwn
Copy link

chaozwn commented Jan 13, 2024

Fixes: #217 #219 Reverts: #213 #218

Hey @b0o @chaozwn, in here I reverted all changes related to on_attach problems and introduced I think cleaner and proper/less hacky fix for it. Can you verify it? I will be really grateful!

I have tested. works well. thanks.

@pmizio pmizio changed the title fix: on_attach called multiple times with separate_diagnostic_server fix: on_attach called multiple times with separate_diagnostic_server Jan 13, 2024
@b0o
Copy link

b0o commented Jan 14, 2024

It works for me too! I like this solution better as well, much cleaner.

@pmizio
Copy link
Owner Author

pmizio commented Jan 14, 2024

@b0o @chaozwn thanks for checking it!

@pmizio pmizio merged commit 094e4f4 into master Jan 14, 2024
34 checks passed
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.

on_attach regression
3 participants