Results 961 comments of Thomas Grainger

It might also be good to hit the explicit str concat `"the user" + repr(u"user") + "did a bad".` and `%r` formatting

this also applies to numeric literals, I just noticed pyupgrade apply this change: ```diff - "hostname": "{}:{}".format(dc, 636), + "hostname": f"{dc}:{636}", ``` but I'd expect `f"{dc}:636"`

I'd probably implement this as two steps, one to replace `six.PY2` with `False` and another to remove unreachable branches

@asottile here's a tool that does automatic dead code removal: https://github.com/coala/coala-bears/pull/2000

it seems they've managed to get it to apply removals to dead code detections https://github.com/jendrikseipp/vulture/issues/25

s/over HSTS/over HTTPS/

@lgarron maybe the header should be changed to: ``` preload, SHA3(`${domain} I have read and understood https://hstspreload.org/#information`) ```

@lgarron 1. no because you can provide a set of SHA3(domain + edu-nonce) and as long as your domain is in the set you win win = preload

Those that know enough to automate it, know not to