Repository navigation
Added whitespace peeve for before attribute access - #36
IanLee1521 wants to merge 1 commit into
Conversation
|
Hello, and thanks for your contribution! Unfortunately we couldn't find an account corresponding to your GitHub username at bugs.python.org (b.p.o). If you don't already have an account at b.p.o, please create one and make sure to add your GitHub username. If you do already have an account at b.p.o then please go there and under "Your Details" add your GitHub username. And in case you haven't already, please make sure to sign the PSF contributor agreement (CLA); we can't legally look at your contribution until you have signed the CLA. Once you have done everything that's needed, please reply here and someone will verify everything is in order. |
|
Oops, hadn't updated my b.p.o profile. Just did! |
|
Have you talked to any of the PEP authors about this change (I have not seen it brought up on python-dev)? You will need one of the PEP authors to okay this before we can merge it. |
|
To be honest, though, I doubt this is a controversial recommendation. The debate (if there is any) would be more "Do we even need to say this?" rather than "Is this duff advice?". |
|
@Rosuav Yes, that's my opinion. PEP 8 isn't a legal document so it doesn't need to spell out every single detail. This just seems unnecessary. I'm going to close this PR, and if any of the other PEP authors want to debate it further, we can reopen it. |
This missing advice came up on PyCQA/pycodestyle#553.