Skip to content

fix missing stdio param - #385

Merged
zbirenbaum merged 1 commit into
masterfrom
missing-command-param
Mar 20, 2025
Merged

fix missing stdio param#385
zbirenbaum merged 1 commit into
masterfrom
missing-command-param

Conversation

@zbirenbaum

Copy link
Copy Markdown
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

zbirenbaum commented Mar 20, 2025

Copy link
Copy Markdown
Owner Author

@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

AntoineGS commented Mar 20, 2025

Copy link
Copy Markdown
Collaborator

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.

@AntoineGS
AntoineGS deleted the missing-command-param branch September 4, 2025 02:34
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