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

Remove legacy z_qos_t #536

Closed

sashacmc opened this issue Jul 12, 2024 · 0 comments

Assignees

Labels

release

Member

sashacmc commented Jul 12, 2024

Describe the release item

We already getters for it and seems we don't need expose it like in this way:

z_sample_qos
include/zenoh-pico/api/primitives.h#L1457
z_qos_t
include/zenoh-pico/api/types.h#L432
z_qos_default
include/zenoh-pico/api/types.h#L457
z_qos_get_congestion_control
include/zenoh-pico/api/types.h#L445
z_qos_get_express
include/zenoh-pico/api/types.h#L452
z_qos_get_priority
include/zenoh-pico/api/types.h#L437

sashacmc added the release label

eclipse-zenoh-bot added this to Zenoh 1.0.0 release

sashacmc changed the title ~~(TBD) Remove legacy z_qos_t~~ Remove legacy z_qos_t

milyin assigned sashacmc

sashacmc mentioned this issue

Remove legacy z_qos_t type #593

Merged

sashacmc closed this as completed

github-project-automation bot moved this to Done in Zenoh 1.0.0 release

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment