-
-
Notifications
You must be signed in to change notification settings - Fork 96
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
docs: mention receive_X_changes in dbus_proxy #1071
base: main
Are you sure you want to change the base?
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.
Thanks! LGTM otherwise. Just a typo and tiny improvements and we can just merge it. 👍
@@ -106,6 +106,9 @@ mod utils; | |||
/// a [`zbus::message::Message`] wrapper, named `<SignalName>`. This wrapper provides type safe | |||
/// access to the signal arguments. It also implements `Deref<Target = Message>` to allow easy | |||
/// access to the underlying [`zbus::message::Message`]. | |||
/// | |||
/// For each property with `emits_changed_signal` = `"true"` or `"invalidates"`, this macro will |
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.
/// For each property with `emits_changed_signal` = `"true"` or `"invalidates"`, this macro will | |
/// For each property with `emits_changed_signal` set to `"true"` (default) or `"invalidates"`, this macro will |
@@ -106,6 +106,9 @@ mod utils; | |||
/// a [`zbus::message::Message`] wrapper, named `<SignalName>`. This wrapper provides type safe | |||
/// access to the signal arguments. It also implements `Deref<Target = Message>` to allow easy | |||
/// access to the underlying [`zbus::message::Message`]. | |||
/// | |||
/// For each property with `emits_changed_signal` = `"true"` or `"invalidates"`, this macro will | |||
/// provide a method names `receive_<property_name>_changed` that creates a [`zbus::PropertyStream`]. |
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.
/// provide a method names `receive_<property_name>_changed` that creates a [`zbus::PropertyStream`]. | |
/// provide a method named `receive_<property_name>_changed` that creates a [`zbus::PropertyStream`] | |
/// for the property. |
Feel free to convert this to a documentation request issue rather than merging this PR.