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
feat: com aux able to receive timestamps #466
feat: com aux able to receive timestamps #466
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #466 +/- ##
=======================================
Coverage 98.14% 98.14%
=======================================
Files 84 84
Lines 6747 6750 +3
=======================================
+ Hits 6622 6625 +3
Misses 125 125 ☔ View full report in Codecov by Sentry. |
|
||
# stay with the old return type to not making a breaking change | ||
if remote_id is not None: | ||
if remote_id and timestamp: | ||
return (msg, remote_id, timestamp) |
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.
That's still a breaking change, or am I missing something?
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.
Please add an additional keyword argument receive_timestamp: bool = False
so that no breaking change occurs
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.
Please add an entry to the what's new then we're good to go :)
8e2008b
to
b34ecec
Compare
@@ -136,6 +136,7 @@ def receive_message( | |||
self, | |||
blocking: bool = True, | |||
timeout_in_s: float = None, | |||
receive_timestamp: bool = False, | |||
) -> Optional[bytes]: |
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.
) -> Optional[bytes]: | |
) -> Optional[bytes | Tuple[bytes, int] | Tuple[bytes, int, float]]: |
@@ -136,6 +136,7 @@ def receive_message( | |||
self, | |||
blocking: bool = True, | |||
timeout_in_s: float = None, | |||
receive_timestamp: bool = False, |
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.
Parameter docstring is missing 🙈
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.
Ouuups, sorry I was trying to be quick so it can be merge before 0.28.0 🚑
No description provided.