-
Notifications
You must be signed in to change notification settings - Fork 751
feat(amazonq): handle reference for inline with language server #6837
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
Conversation
|
| session.firstCompletionDisplayLatency, | ||
| ], | ||
| } | ||
| ReferenceInlineProvider.instance.setInlineReference( |
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.
I'm not entirely familiar with the flow but is it something like this:
- user gets inline suggestion
- it may have a reference attached
- if it does have a reference then show it as a code lens
- if they click the codelens to add a code reference it adds it in the "Code reference log" panel and opens it up
Is that mostly correct?
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.
yes that is right, and I realized that I missed the logging reference on acceptance piece, just added it. Thanks for catching!
|
Also, is this dependent on any flare changes? |
No, flare already has references support since beginning |
…6837) ## Problem missing log references for inline with language server ## Solution add them --- - Treat all work as PUBLIC. Private `feature/x` branches will not be squash-merged at release time. - Your code changes must meet the guidelines in [CONTRIBUTING.md](https://github.com/aws/aws-toolkit-vscode/blob/master/CONTRIBUTING.md#guidelines). - License: I confirm that my contribution is made under the terms of the Apache 2.0 license.
Problem
need to log reference for inline with language server
Solution
feature/xbranches will not be squash-merged at release time.