Skip to content

CHANGE @W-21910054@ default include suggestions - #369

Merged
nikhil-mittal-165 merged 1 commit into
devfrom
default-include-suggestions-on
Aug 19, 2026
Merged

CHANGE @W-21910054@ default include suggestions#369
nikhil-mittal-165 merged 1 commit into
devfrom
default-include-suggestions-on

Conversation

@nikhil-mittal-165

Copy link
Copy Markdown
Contributor

default include suggestions for apex guru suggestions popping up by default

@git2gus

git2gus Bot commented Aug 19, 2026

Copy link
Copy Markdown

Git2Gus App is installed but the .git2gus/config.json doesn't have right values. You should add the required configuration.

@aruntyagiTutu aruntyagiTutu left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

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.

@nikhil-mittal-165
nikhil-mittal-165 merged commit 0d402ba into dev Aug 19, 2026
9 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