Disables update on startup if autoupdate is off#1
Merged
ayagmar merged 1 commit intoayagmar:masterfrom Feb 16, 2026
Merged
Conversation
Added getAutoUpdateConfig to retrieve configuration and check if auto-update is enabled before starting the timer.
Owner
|
Thanks! |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Currently, even when auto-update is disabled via
/extensions auto-update never, the extension still runs package checks whenpiis opened, greatly increasing loading time.This happens because
startAutoUpdateTimer()is called unconditionally during the session bootstrap.This PR introduces a check against the auto-update config before starting the timer and only starts the timer if auto-update is explicitly enabled and has a valid interval.
P.S. I believe that this is not the best way to do this. Ideally there should be a setting/flag that disables the updates on start-up regardless of the auto-update setting. But my solution works fine for me so I won't pursue this further.