Skip to content
This repository has been archived by the owner on May 17, 2024. It is now read-only.

feat: UDP listener + connector #80

Merged
merged 9 commits into from
Sep 2, 2023
Merged

feat: UDP listener + connector #80

merged 9 commits into from
Sep 2, 2023

Conversation

lino-levan
Copy link
Contributor

Unfortunately UDP is unstable, but I still think this example is valuable.

@lino-levan lino-levan mentioned this pull request Jun 24, 2023
Closed
44 tasks
@kt3k kt3k mentioned this pull request Jun 26, 2023
Closed
@iuioiua
Copy link
Contributor

iuioiua commented Sep 1, 2023

Can you please run deno fmt?

@lino-levan
Copy link
Contributor Author

Not sure this is mergable given the api is unstable and likely to change. I'll fmt when I get to my hotel.

@iuioiua
Copy link
Contributor

iuioiua commented Sep 2, 2023

@kt3k, is this something worth adding?

kt3k
kt3k approved these changes Sep 2, 2023
View reviewed changes
Copy link
Member

@kt3k kt3k left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the ping. I think it's still useful with the current state. LGTM

Let's update when the API changed.

@kt3k kt3k merged commit 55a8132 into denoland:main Sep 2, 2023
3 checks passed
@lino-levan lino-levan deleted the feat-udp branch October 7, 2023 18:30
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@kt3k kt3k kt3k approved these changes

@lucacasonato lucacasonato Awaiting requested review from lucacasonato lucacasonato is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@lino-levan @iuioiua @kt3k