-
-
Notifications
You must be signed in to change notification settings - Fork 718
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
Make ToPickle
a Generic
#8097
Make ToPickle
a Generic
#8097
Conversation
Unit Test ResultsSee test report for an extended history of previous test failures. This is useful for diagnosing flaky tests. 18 files - 2 18 suites - 2 9h 30m 18s ⏱️ - 2h 5m 11s For more details on these failures, see this check. Results for commit 0d10798. ± Comparison against base commit 4f30abc. This pull request removes 1 and adds 15 tests. Note that renamed tests count towards both.
♻️ This comment has been updated with latest results. |
distributed/protocol/serialize.py
Outdated
return isinstance(other, type(self)) and other.data == self.data | ||
|
||
def __ne__(self, other): | ||
def __ne__(self, other: object) -> bool: | ||
return not (self == other) |
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.
out of scope: fairly sure this method is redundant with __eq__
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.
Good point, didn't even pay attention to this. Dropped __ne__
as a drive-by fix.
Co-authored-by: crusaderky <crusaderky@gmail.com>
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.
Good to merge as soon as CI finishes
Extracted from #8096
pre-commit run --all-files