← Back to team overview

yahoo-eng-team team mailing list archive

[Bug 1609848] Re: Un-ignore PEP8 rules in tox

 

Seems that all of the rules are being ignored for good (or at least
agreed-upon) reasons.

See https://review.openstack.org/#/c/351253/

** Changed in: nova
       Status: New => Invalid

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to OpenStack Compute (nova).
https://bugs.launchpad.net/bugs/1609848

Title:
  Un-ignore PEP8 rules in tox

Status in OpenStack Compute (nova):
  Invalid

Bug description:
  tox.ini contains a number of entries in the 'ignore' list for flake8,
  allowing developers to kick certain minor infractions down the road,
  resulting in creeping technical debt.

  Wherever feasible, these ignore entries should be weeded out and their
  respective issues resolved in the codebase.

  To work this bug:
  0) Pull the latest code.  This will minimize painful rebasing.
  1) Edit tox.ini.  Remove one or more tokens from the 'ignore' definition under the [flake8] section.
  2) Run tox -e pep8
  3) Resolve all of the errors
  4) Propose up your change, referencing this bug report's ID with the Partial-Bug tag.

To manage notifications about this bug go to:
https://bugs.launchpad.net/nova/+bug/1609848/+subscriptions


References