Skip to content

fix missing stdio param #385

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

Merged
merged 1 commit into from
Mar 20, 2025
Merged

fix missing stdio param #385

merged 1 commit into from
Mar 20, 2025

Conversation

zbirenbaum
Copy link
Owner

add --stdio to end of command. Having some trouble with getting completions to appear even though executable is running so may have to revert #372

@zbirenbaum
Copy link
Owner Author

zbirenbaum commented Mar 20, 2025

@AntoineGS Do you use the autosuggestion or copilot-cmp at all. I'm not getting completions even with this fix

It's working but sometimes not on the first buffer. If the attach pr fixes let's prioritize that one

@zbirenbaum zbirenbaum merged commit acb0545 into master Mar 20, 2025
@AntoineGS
Copy link
Collaborator

AntoineGS commented Mar 20, 2025

Hmm it seems to work on the first buffer but only after writing some text.
I'll see if that's a new behavior or not and investigate.

Edit: It seems to be the same across version, older and the ones in my PRs.
Sometimes is takes a few keys to trigger, sometimes it works right away, it might be that the server takes a few seconds to start sending suggestions.

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.

2 participants