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.

Sign up for GitHub

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

Jump to bottom

Implement and test transport::ZenohUTransport #51

Closed
gregmedd opened this issue May 21, 2024 · 3 comments · Fixed by #69 or #65
Closed

Implement and test transport::ZenohUTransport #51

gregmedd opened this issue May 21, 2024 · 3 comments · Fixed by #69 or #65
Assignees
@gregmedd @MarkStokes71
Milestone
alpha.4

Comments

@gregmedd
Copy link
Contributor

gregmedd commented May 21, 2024 •
edited
Loading

Make it work!

Use the C++ Zenoh library to implement a UTransport under the new API. Test that it works.
@gregmedd gregmedd converted this from a draft issue May 21, 2024
@sashacmc
Copy link
Contributor

@gregmedd please assign it to me

@sashacmc sashacmc mentioned this issue May 29, 2024
Closed
@gregmedd gregmedd moved this from Backlog to In progress in C++ uProtocol Development May 30, 2024
@gregmedd gregmedd added this to the alpha.2 milestone May 30, 2024
@gregmedd gregmedd self-assigned this Jul 8, 2024
@MarkStokes71
Copy link
Contributor

I will start reviewing this.

@gregmedd gregmedd assigned MarkStokes71 and unassigned sashacmc Jul 15, 2024
@gregmedd gregmedd modified the milestones: alpha.2, alpha.4 Jul 30, 2024
This was linked to pull requests Jul 30, 2024
Fix unittest and code cleanup #69
Merged
Feature/1.5.8/zenoh utransport impl #65
Merged
@gregmedd
Copy link
Contributor Author

gregmedd commented Aug 14, 2024 •
edited
Loading

The implementation appears to be passing all existing use case testing. Adding more UT coverage is tracked in #95 and #79. I believe this task can now be marked as completed.

@github-project-automation github-project-automation bot moved this from In progress to Done in C++ uProtocol Development Aug 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@gregmedd gregmedd

@MarkStokes71 MarkStokes71

Labels
None yet
Projects
C++ uProtocol Development
Status: Done
Milestone
alpha.4
3 participants
@gregmedd @MarkStokes71 @sashacmc