Skip to content

Delete pulled diagnostics when the source registration is unregistered#46105

Merged
reflectronic merged 1 commit intozed-industries:mainfrom
reflectronic:pull-diagnostics-unregistration
Jan 5, 2026
Merged

Delete pulled diagnostics when the source registration is unregistered#46105
reflectronic merged 1 commit intozed-industries:mainfrom
reflectronic:pull-diagnostics-unregistration

Conversation

@reflectronic
Copy link
Member

@reflectronic reflectronic commented Jan 5, 2026

Additionally, fix a race condition where we'd still insert diagnostics from a document or workspace pull even if the registration had been unregistered in the time since the request was issued.

And, as a bonus: when a new pull diagnostics registration is added, issue document pulls immediately.

This should fix regressions with basedpyright caused by #43703.

Release Notes:

  • N/A

@cla-bot cla-bot bot added the cla-signed The user has signed the Contributor License Agreement label Jan 5, 2026
@reflectronic reflectronic merged commit 8ba030a into zed-industries:main Jan 5, 2026
25 checks passed
@reflectronic
Copy link
Member Author

/cherry-pick preview

github-actions bot pushed a commit that referenced this pull request Jan 5, 2026
#46105)

Additionally, fix a race condition where we'd still insert diagnostics
from a document or workspace pull even if the registration had been
unregistered in the time since the request was issued.

And, as a bonus: when a new pull diagnostics registration is added,
issue document pulls immediately.

This should fix regressions with basedpyright caused by
#43703.

Release Notes:

- N/A
zed-zippy bot added a commit that referenced this pull request Jan 5, 2026
#46105) (cherry-pick to preview) (#46117)

Cherry-pick of #46105 to preview

----
Additionally, fix a race condition where we'd still insert diagnostics
from a document or workspace pull even if the registration had been
unregistered in the time since the request was issued.

And, as a bonus: when a new pull diagnostics registration is added,
issue document pulls immediately.

This should fix regressions with basedpyright caused by
#43703.

Release Notes:

- N/A

Co-authored-by: John Tur <john-tur@outlook.com>
LivioGama pushed a commit to LivioGama/zed that referenced this pull request Jan 20, 2026
zed-industries#46105)

Additionally, fix a race condition where we'd still insert diagnostics
from a document or workspace pull even if the registration had been
unregistered in the time since the request was issued.

And, as a bonus: when a new pull diagnostics registration is added,
issue document pulls immediately.

This should fix regressions with basedpyright caused by
zed-industries#43703.

Release Notes:

- N/A
LivioGama pushed a commit to LivioGama/zed that referenced this pull request Jan 20, 2026
zed-industries#46105)

Additionally, fix a race condition where we'd still insert diagnostics
from a document or workspace pull even if the registration had been
unregistered in the time since the request was issued.

And, as a bonus: when a new pull diagnostics registration is added,
issue document pulls immediately.

This should fix regressions with basedpyright caused by
zed-industries#43703.

Release Notes:

- N/A
LivioGama pushed a commit to LivioGama/zed that referenced this pull request Feb 15, 2026
zed-industries#46105)

Additionally, fix a race condition where we'd still insert diagnostics
from a document or workspace pull even if the registration had been
unregistered in the time since the request was issued.

And, as a bonus: when a new pull diagnostics registration is added,
issue document pulls immediately.

This should fix regressions with basedpyright caused by
zed-industries#43703.

Release Notes:

- N/A
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla-signed The user has signed the Contributor License Agreement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant