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.

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

Improve completion of Copilot keywords #384

Merged
merged 2 commits into from
Aug 6, 2024

Conversation

guill
Copy link
Contributor

@guill guill commented Aug 3, 2024

Previously, completion in nvim-cmp would only happen when typing the @ or / character. There
was no way to type a partial command (e.g. /COPILOT_G) and then autocomplete.

This PR adds a keyword pattern to allow this sort of completion in a case-insensitive way.

If the previous behavior was an intentional design decision, feel free to close this PR. (I changed
it for my own usage, but figured other people might like it as well.)

@jellydn jellydn requested a review from deathbeam August 4, 2024 08:30
guill and others added 2 commits August 4, 2024 16:30
Instead of only triggering completion with '/' or '@', completion can
also be manually triggered.
@jellydn jellydn force-pushed the feat/improved_cmp branch from cbd47e5 to 3324c70 Compare August 4, 2024 08:30
@jellydn
Copy link
Contributor

jellydn commented Aug 4, 2024

Thanks @guill. I don't use this feature much so let's see @deathbeam's view on this approach.

@deathbeam
Copy link
Collaborator

I dont use it either (as I dont use nvim-cmp) but looks good to me

@jellydn jellydn merged commit 34dc3cf into CopilotC-Nvim:canary Aug 6, 2024
1 check passed
@jellydn
Copy link
Contributor

jellydn commented Aug 6, 2024

@all-contributors add @guill for code.

Copy link
Contributor

@jellydn

I've put up a pull request to add @guill! 🎉

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.

3 participants