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
Use black to format code #146
Conversation
b46ea80
to
a236ad3
Compare
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.
Oooh. The dawn of a new age. Time to deprecate y/psg ?
I reviewed it mainly because I was interested in what it looks like. Just a few weird things, but by definition they are all nits 😸
a236ad3
to
32aa1c4
Compare
@@ -89,22 +86,18 @@ def _convert_from_thrift_endpoint(self, thrift_endpoint): | |||
""" | |||
ipv4 = None | |||
ipv6 = None | |||
port = struct.unpack('H', struct.pack('h', thrift_endpoint.port))[0] | |||
port = struct.unpack("H", struct.pack("h", thrift_endpoint.port))[0] |
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.
this reminds me of eslint in reverse
32aa1c4
to
9372f29
Compare
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.
Ship it!
9372f29
to
c6c4379
Compare
Similar to what we're doing in other repos, let's use black to format the code to ensure a consistent style