-
Notifications
You must be signed in to change notification settings - Fork 142
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
Remove keyexpr with_parameters #979
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Honestly, I would prefer to keep this API, because it can be more convenient to chain calls, for example if you retrieve the keyexpr from another object.
But I've no experience about API use, so my opinion doesn't count a lot.
Co-authored-by: Joseph Perez <joperez@hotmail.fr>
It's more about removing unnecessary API that needs to be supported in the various bindings. I don't see much difference in the actual API usage in terms of complexity (at least as it was used in the zenoh codebase) and it will remove the need of eexposing the method on all other languages. |
Fair point, I will remove it from the Python API so 🙃 |
LGTM |
Related to #974