🌐 US-Proxy
class="logged-out env-production page-responsive" style="word-wrap: break-word;" >
Skip to content

Improve responsiveness to cmake changes made by vendors - #4908

Merged
Hannia Valera (hanniavalera) merged 2 commits into
microsoft:mainfrom
JDESTM:improve-responsiveness-to-cmake-changes-made-by-vendors
Apr 27, 2026
Merged

Hannia Valera (hanniavalera) merged 2 commits into
microsoft:mainfrom
JDESTM:improve-responsiveness-to-cmake-changes-made-by-vendors

Conversation

@JDESTM

Copy link
Copy Markdown
Contributor

This changes improve responsiveness to cmake.cmakePath configuration changes made by Vendors during the extension initialization phase

The following changes are proposed:

  • subscribe to onDidChangeConfiguration in the wait loop in addition to setTimeout
  • minor vendor friendly name update

@hanniavalera

Copy link
Copy Markdown
Contributor

Julien D (@JDESTM) Thanks for putting this up, I appreciate the ongoing cooperation!

@hanniavalera

Copy link
Copy Markdown
Contributor

Had to address some minor lint errors + wanted to get you in the CHANGELOG :) let me know if these changes make sense to you! Julien D (@JDESTM)

@JDESTM Julien D (JDESTM) left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

My bad, I completely forgot to invoke the linter before pushing it, thanks for having fixed this 👍

Comment thread CHANGELOG.md Outdated
@JDESTM

Copy link
Copy Markdown
Contributor Author

@microsoft-github-policy-service agree company="STMicroelectronics International N.V."

Signed-off-by: Julien Dehaudt <91720930+JDESTM@users.noreply.github.com>
Signed-off-by: Julien Dehaudt <91720930+JDESTM@users.noreply.github.com>
@JDESTM
Julien D (JDESTM) force-pushed the improve-responsiveness-to-cmake-changes-made-by-vendors branch from ccc4db1 to a9e387b Compare April 24, 2026 19:19
@hanniavalera
Hannia Valera (hanniavalera) merged commit 38ff27a into microsoft:main Apr 27, 2026
4 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.

2 participants