-
Notifications
You must be signed in to change notification settings - Fork 1.7k
Enable autocomplete by default for JetBrains users #4836
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
🦋 Changeset detectedLatest commit: c3b94c5 The changes in this PR will be included in the next version bump. Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
| } | ||
| // Auto-enable autocomplete by default, but disable for JetBrains IDEs | ||
| // JetBrains users can manually enable it if they want to test the feature | ||
| // Auto-enable autocomplete by default |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
WARNING: Unused import left behind
The getKiloCodeWrapperProperties import on line 12 is no longer used after removing the JetBrains-specific checks. This dead import should be removed to keep the code clean and avoid lint warnings.
Code Review SummaryStatus: 2 Issues Found | Recommendation: Address before merge Overview
Issue Details (click to expand)CRITICAL
WARNING
Files Reviewed (3 files)
|
c0cdbbf to
c3b94c5
Compare
| @@ -0,0 +1,3 @@ | |||
| "kilocode-jetbrains": minor | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
CRITICAL: Invalid changeset format - missing --- delimiters
The changeset file is missing the required YAML frontmatter delimiters. Without these, the changeset won't be parsed correctly by the changesets tool.
| "kilocode-jetbrains": minor | |
| --- | |
| "kilocode-jetbrains": minor | |
| --- |
Summary
This PR removes the JetBrains-specific gate that was preventing autocomplete from being enabled by default.
Changes
Background
Previously, autocomplete ( and ) was disabled by default for JetBrains IDEs. This was a conservative approach while the feature was being tested. Now that the feature is stable, we're enabling it by default for all users, including JetBrains users.