StarlingX

  1. Bug #1856585
  2. Comment #6

Comment 6 for bug 1856585

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to update (r/stx.3.0)

Reviewed: https://review.opendev.org/702942
Committed: https://git.openstack.org/cgit/starlingx/update/commit/?id=ecd5a624e22d1a81d44c84bf1e0192dd3af3ad0d
Submitter: Zuul
Branch: r/stx.3.0

commit ecd5a624e22d1a81d44c84bf1e0192dd3af3ad0d
Author: Al Bailey <email address hidden>
Date: Mon Dec 16 08:48:14 2019 -0600

    Fix zuul tox flake8 failures from newer version of hacking

    A newer version of hacking was released which enables additional
    checks and is breaking flake8 and zuul.

    Hacking has now been clamped at 2.0.0 in test-requirements.txt
    Added the same for patch-alarm, although it was not failing.

    The new checks that are fixed are:
     H106: Don't put vim configuration in source files
     W605 invalid escape sequence
     F632 use ==/!= to compare str, bytes, and int literals
     F841 local variable is assigned to but never used

    The following flake8 extension check has been removed:
     H904 Delay string interpolations at logging calls

    Partial-Bug: 1856585
    Change-Id: I8bdc94539f4a6ce243c4e30efc07eed3bad1ce62
    Signed-off-by: Al Bailey <email address hidden>
    (cherry picked from commit 5d2da5d041028afa916149da660809ca6fc4fb73)