-
Notifications
You must be signed in to change notification settings - Fork 140
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 internal APIs from public interface #806
Comments
Consider using |
|
All functionality which is needed by some external crates but should not be part of public API should be moved into traits available through "internal" mod. Here is an example: #1046 |
from @Mallets:
|
Seems that |
Describe the release item
We should not expose internal APIs or at least should explicitly mark them as internal in the documentation (e.g. APIs needed for plugin support only)
The list of APIs to be checked
keyexpr -> get_nonwild_prefix
keyexpr -> is_wild
keyexpr -> chunks
keyexpr -> strip_prefix
publication.rs: Priority::MIN, Priority::MAX, Priority::NUM
WhatAmI matcherZenohId -> size, max_size, rand, into_keyexpr, to_le_bytes
The text was updated successfully, but these errors were encountered: