Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

feat(listener): added windows pipe connection support #409

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

feat(listener): added windows pipe connection support #409

wants to merge 3 commits into from

Conversation

xlango
Copy link

@xlango xlango commented Jun 13, 2024

added windows pipe connection support

@xlango
Copy link
Author

xlango commented Jun 14, 2024

@mochi-co Please approve, I have verified myself

@mochi-co
Copy link
Collaborator

Hi @xlango - looks like the build fails with the following error:

examples/windows_pipe/main.go#L37 undefined: listeners.NewWindowsPipe
Process completed with exit code 1.

Thanks

@coveralls
Copy link

Pull Request Test Coverage Report for Build 11487272586

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-13.2%) to 85.394%
Totals Coverage Status
Change from base Build 11487262495: -13.2%
Covered Lines: 6168
Relevant Lines: 7223
💛 - Coveralls

# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@xlango @mochi-co @coveralls