vscode-cmake-tools
vscode-cmake-tools copied to clipboard
Add option to disable kit scan by default
This change addresses #1461
This change adds an option to disable kit scan by default.
The following changes are proposed:
- Sets kit to Unspecified if enable kit scan is set to false, cmake presets is not being used, and no kit was already selected while enable kit scan was set to true.
[heart] Ion Todirel reacted to your message:
From: Fatima Qarni @.> Sent: Tuesday, February 13, 2024 5:42:08 PM To: microsoft/vscode-cmake-tools @.> Cc: Subscribed @.***> Subject: [microsoft/vscode-cmake-tools] Add option to disable kit scan by default (PR #3585)
This change resolves #[[1461]] This change adds an option to disable kit scan by default.
The following changes are proposed:
- Sets kit to Unspecified if enable kit scan is set to false, and no kit was already selected while enable kit scan was set to true.
You can view, comment on, or merge this pull request online at:
https://github.com/microsoft/vscode-cmake-tools/pull/3585
Commit Summary
- b1b0a9chttps://github.com/microsoft/vscode-cmake-tools/pull/3585/commits/b1b0a9c0b86c91dfd0e22addf2d2ae96ce553d5d Adds option to disable kit scan by default
- 48983e4https://github.com/microsoft/vscode-cmake-tools/pull/3585/commits/48983e44a32d7f8d28fd47ddc9459638919f77e1 Remove unused comment
- 548217ahttps://github.com/microsoft/vscode-cmake-tools/pull/3585/commits/548217a25ddd39d0e046620d1270283f3e77e732 remove newline
File Changes
(5 fileshttps://github.com/microsoft/vscode-cmake-tools/pull/3585/files)
- M package.jsonhttps://github.com/microsoft/vscode-cmake-tools/pull/3585/files#diff-7ae45ad102eab3b6d7e7896acd08c427a9b25b346470d7bc6507b6481575d519 (6)
- M package.nls.jsonhttps://github.com/microsoft/vscode-cmake-tools/pull/3585/files#diff-9b29d8b47a74097a04da353b72a8632d485d658282457b97b4caf26575a42dd0 (1)
- M src/config.tshttps://github.com/microsoft/vscode-cmake-tools/pull/3585/files#diff-c3095d5010e65c52737a98a5d618ea24049ebe90c8470752426081d70ed6e012 (8)
- M src/extension.tshttps://github.com/microsoft/vscode-cmake-tools/pull/3585/files#diff-04bba6a35cad1c794cbbe677678a51de13441b7a6ee8592b7b50be1f05c6f626 (7)
- M test/unit-tests/config.test.tshttps://github.com/microsoft/vscode-cmake-tools/pull/3585/files#diff-ec8e8e049c8708804b00eb224b09273fed88192d0851596d7d309963ec47fd75 (3)
Patch Links:
- https://github.com/microsoft/vscode-cmake-tools/pull/3585.patch
- https://github.com/microsoft/vscode-cmake-tools/pull/3585.diff
— Reply to this email directly, view it on GitHubhttps://github.com/microsoft/vscode-cmake-tools/pull/3585, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ADUIVMBOWRPCLBCPJUKW22TYTOQXBAVCNFSM6AAAAABDG5RUNWVHI2DSMVQWIX3LMV43ASLTON2WKOZSGEZTEOBUGU3DOMI. You are receiving this because you are subscribed to this thread.Message ID: @.***>
@qarni Oh, one other thing, when we add new features, we make sure to make a CHANGELOG.md update, could you do that too?
When was the PR released?
@frankie-zeng It has not been released yet, but it will be very soon. Thank you for pinging on this issue!
@gcampbell-msft thanks, I realy need this feature!!, see it in 1.18
@frankie-zeng It should be officially released now in the most recent official version. THanks!