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

Add named pipes for windows support #8

Merged
merged 1 commit into from
Sep 8, 2024
Merged

Conversation

Tiggilyboo
Copy link

First of all, thanks for the great little wrapper. I mainly use C# for work using Windows, and love helix - came across this project through a reddit post - either way, here is the fix for using this project on windows.

I don't write much Rust code, but here's what I've got working - open to any cleanup / tips if you see anything weird.

  • Fixes Support Windows and named pipes #7
  • Tested it on my work machine, seems to pull in references and autocompletions.
  • Tested as well on my linux box, still works there too!

@SofusA
Copy link
Owner

SofusA commented Sep 8, 2024

This is awesome! Thanks for working on this.

If you did not know, diagnostics does not work in helix yet. I have a pull request for supporting pull diagnostics. You can use my fork until then.

@SofusA SofusA merged commit a5ee336 into SofusA:main Sep 8, 2024
@SofusA SofusA mentioned this pull request Sep 9, 2024
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.

Support Windows and named pipes
2 participants