-
Notifications
You must be signed in to change notification settings - Fork 325
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
Support llmGenerated
property on CodeAction
#1557
Conversation
Looking at the implementation now another boolean here feels wrong. I think we should follow the same approach as we did with other literals and add a |
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.
Great work. Only some minor wording tweaks.
Let's wait until we have the spec right and then we finish up the coding. Is that OK for you? |
Sure, I've updated the spec. Let me know if it looks good:) |
I've added the |
As far as I can tell this is enabled for the extension (in product.json). Marking the extension side verified (microsoft/vscode-python#24068) |
@StellaHuang95 nice work! |
Addresses #1555
llmGeneratedSupport
toCodeActionClientCapabilities
.llmGenerated
toCodeAction
.protocolConverter
andcodeConverter
.enabledApiProposals
inpackage.json
and the corresponding.d.ts
file.