yahoo-eng-team team mailing list archive
-
yahoo-eng-team team
-
Mailing list archive
-
Message #54853
[Bug 1611400] Re: pep8 job failing with F821 in test_l3
Reviewed: https://review.openstack.org/352941
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=a200f4a7b670abd7f6fc59847782719057bfe6ef
Submitter: Jenkins
Branch: master
commit a200f4a7b670abd7f6fc59847782719057bfe6ef
Author: Ihar Hrachyshka <ihrachys@xxxxxxxxxx>
Date: Tue Aug 9 16:32:33 2016 +0200
pep8: fixed F821 violation in a unit test
The _ symbol should have been imported explicitly. Actually, there is no
use case for translatable messages in unit tests, it's just a waste of
translator time to mark it as such. The patch removes the translation
marker for good.
Change-Id: I76bef710a1030dcb4eb35778ebf44ed600016f17
Closes-Bug: #1611400
** Changed in: neutron
Status: In Progress => Fix Released
--
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to neutron.
https://bugs.launchpad.net/bugs/1611400
Title:
pep8 job failing with F821 in test_l3
Status in neutron:
Fix Released
Bug description:
New flake8 was released today, and probably broke neutron pep8 job
with:
pep8 runtests: commands[2] | flake8
./neutron/tests/unit/extensions/test_l3.py:2931:19: F821 undefined name '_'
msg = _("Failure mocking...")
^
To manage notifications about this bug go to:
https://bugs.launchpad.net/neutron/+bug/1611400/+subscriptions
References