Skip to content

Reduce event handlers in nb controller ctor#227333

Merged
DonJayamanne merged 2 commits intomainfrom
don/issue204416
Sep 2, 2024
Merged

Reduce event handlers in nb controller ctor#227333
DonJayamanne merged 2 commits intomainfrom
don/issue204416

Conversation

@DonJayamanne
Copy link
Contributor

Fixes #204416

@DonJayamanne DonJayamanne marked this pull request as ready for review September 2, 2024 04:47
@DonJayamanne DonJayamanne marked this pull request as draft September 2, 2024 04:47
@DonJayamanne DonJayamanne changed the title Recuce event handlers in nb controller ctor Reduce event handlers in nb controller ctor Sep 2, 2024
@DonJayamanne DonJayamanne marked this pull request as ready for review September 2, 2024 06:08
@DonJayamanne DonJayamanne enabled auto-merge (squash) September 2, 2024 06:09
@DonJayamanne DonJayamanne merged commit 519655d into main Sep 2, 2024
@DonJayamanne DonJayamanne deleted the don/issue204416 branch September 2, 2024 06:09
@vs-code-engineering vs-code-engineering bot added this to the September 2024 milestone Sep 2, 2024
@vs-code-engineering vs-code-engineering bot locked and limited conversation to collaborators Oct 17, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Too many onDidChangeSelectedNotebooks subscribers leads to new kernels not being able to accept notebook association

2 participants