CHANGE @W-21910054@ default include suggestions - #369
Conversation
|
Git2Gus App is installed but the |
aruntyagiTutu
left a comment
There was a problem hiding this comment.
Small, focused change matching the PR description and W-21910054: flips the codeAnalyzer.includeSuggestions default from false to true, consistently updated in both package.json's setting declaration and settings.ts's getConfiguration(...).get() fallback — so the two stay in sync (a common source of bugs when only one side is updated).
One concern per PR, commit title follows convention, no unrelated changes mixed in. test/stubs.ts's getIncludeSuggestionsReturnValue was already defaulted to true, so existing tests aren't affected by this flip. No correctness or performance concerns for a static default-value change.
default include suggestions for apex guru suggestions popping up by default