[issue34693] PEPping distutils/core.py

[John Joyce]

New submission from John Joyce :

In distutils/core.py ...

Line 227 is over-indented, inconsistent with the rest of the file.
Line 226 is over-indented, inconsistent with the rest of the file.

Line 114 is over-indented by 2 characters, inconsistent with the rest of the 
file.
else:
raise SystemExit("error in %s setup command: %s" % \
  (attrs['name'], msg))


Line 113 backslash should not be needed to continue the line between brackets.

Line 102 there is an extra space before the "not" keyword.
if 'script_args'  not in attrs:


Line 57 could use 1 more blank line above it.

Line 33 'script' is assigned but not used.

Line 32 could use 1 more blank line above it.


run_setup and setup had space before (

--
components: Distutils
messages: 325433
nosy: dangerwillrobinsondanger, dstufft, eric.araujo
priority: normal
pull_requests: 8752
severity: normal
status: open
title: PEPping distutils/core.py
type: compile error
versions: Python 3.8

___
Python tracker 
<https://bugs.python.org/issue34693>
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue34693] PEPping distutils/core.py

[John Joyce]

John Joyce  added the comment:

Oh, I see. I did not realize that tidying would not be accepted. That explains 
a lot. As it offended linters based on PEPs I chose that as the best match. It 
is pedantic, but many linting actions are not far from compiling... they are 
generally parsing to determine PEP compliance.

--

___
Python tracker 
<https://bugs.python.org/issue34693>
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com