On 01/09/2021 20:04, Aaron Conole wrote:

Stephen Hemminger [off-list ref] writes:

quoted

On Wed, 01 Sep 2021 08:31:27 -0400
Aaron Conole [off-list ref] wrote:

quoted

  $ flake8 ./usertools/dpdk-devbind.py | sed 's@./usertools/dpdk-devbind.py[:0-9]* @@' | sort -u
  E128 continuation line under-indented for visual indent
  E302 expected 2 blank lines, found 1
  E305 expected 2 blank lines after class or function definition, found 1
  E501 line too long (105 > 79 characters)
  E501 line too long (80 > 79 characters)
  E501 line too long (82 > 79 characters)
  E501 line too long (83 > 79 characters)
  E501 line too long (84 > 79 characters)
  E501 line too long (85 > 79 characters)
  E501 line too long (86 > 79 characters)
  E501 line too long (91 > 79 characters)

Current practice on many projects has allowed lines up to 100 characters.

It is probably okay to run with '--ignore=E501' (which will squelch the
character limit).

I added # noqa : E501 in the appropriate places, as we can't depend on folks using --ignore=E501.