Skip to content

fix: memory leak in accessibilty signal scheduler #263147

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
Merged
meganrogge merged 1 commit into from
Aug 25, 2025
Merged

fix: memory leak in accessibilty signal scheduler #263147

meganrogge merged 1 commit into from
Aug 25, 2025

Conversation

@SimonSiefke
Copy link
Contributor

@SimonSiefke SimonSiefke commented Aug 24, 2025

Registering the RunOnceScheduler as a disposable.
@Yoyokrazy Yoyokrazy assigned meganrogge and unassigned Yoyokrazy Aug 25, 2025
@Yoyokrazy Yoyokrazy requested a review from meganrogge August 25, 2025 16:17
@meganrogge
Copy link
Contributor

meganrogge commented Aug 25, 2025

Thanks!
@vs-code-engineering vs-code-engineering bot added this to the August 2025 milestone Aug 25, 2025
@meganrogge meganrogge enabled auto-merge (squash) August 25, 2025 18:12
@meganrogge meganrogge merged commit ced2626 into microsoft:main Aug 25, 2025
17 checks passed
osortega pushed a commit that referenced this pull request Aug 26, 2025
@SimonSiefke @osortega
fix: memory leak in accessibilty signal scheduler (#263147)
2f702fe
@vs-code-engineering vs-code-engineering bot locked and limited conversation to collaborators Oct 9, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

triage-needed

4 participants

@SimonSiefke @meganrogge @TylerLeonhardt @Yoyokrazy