black introduces PEP 8 violation: operators should go at beginning of lines when possible

[njsmith]

 @contextmanager
 def ki_manager(deliver_cb, restrict_keyboard_interrupt_to_checkpoints):
     if (
-        threading.current_thread() != threading.main_thread()
-        or signal.getsignal(signal.SIGINT) != signal.default_int_handler
+        threading.current_thread() != threading.main_thread() or
+        signal.getsignal(signal.SIGINT) != signal.default_int_handler
     ):

Ok, technically PEP 8 waffles a bit b/c it used to recommend the other version. But these days it says that operator-at-beginning "usually results in more readable code" and "For new code [operator-at-beginning] style is suggested".

[ambv]

Oh, I never noticed PEP 8's change in direction here. This is a relatively new change, Knuth-inspired, no less:

python/peps@c59c437

I can make this change in Black, although it's a surprising one to me. What do you think about this personally?

[njsmith]

This was always one of my biggest wtf's with PEP 8, so I'm excited they changed it :-)

[ambv]

Yay! Thanks, @autophagy.