Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Update inline completion handling based on modelVersion changes. Also apply to ghost text #239582

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

Update inline completion handling based on modelVersion changes. Also apply to ghost text #239582

wants to merge 5 commits into from
+108 −112

Conversation

benibenj
Copy link
Contributor

@benibenj benibenj commented Feb 4, 2025 •
edited
Loading

Refactor inline completion logic to utilize updated edit methods and add equality checks for edit objects.
cc: @hediet

@benibenj benibenj self-assigned this Feb 4, 2025
@benibenj benibenj enabled auto-merge (squash) February 4, 2025 10:03
@vs-code-engineering vs-code-engineering bot added this to the February 2025 milestone Feb 4, 2025
aeschli
aeschli previously approved these changes Feb 4, 2025
View reviewed changes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@aeschli aeschli aeschli left review comments

At least 1 approving review is required to merge this pull request.

Assignees

@benibenj benibenj

Labels
None yet
Projects
None yet
Milestone
February 2025
Development

Successfully merging this pull request may close these issues.
2 participants
@benibenj @aeschli