Issue #1517
latest version of pep8 (part of flake8) doesn't like our order of imports
Start date:
Due date:
Estimated time:
Severity:
2. Medium
Version:
Platform Release:
2.8.0
OS:
Triaged:
Yes
Groomed:
No
Sprint Candidate:
No
Tags:
Pulp 2
Sprint:
Quarter:
Description
When I run ./run-tests.py in the latest vagrant environment I see the following output:
./pulp-dev.py:15:1: E402 module level import not at top of file
./nodes/test/nodes_tests/test_admin_extensions.py:10:1: E402 module level import not at top of file
./nodes/test/nodes_tests/test_admin_extensions.py:11:1: E402 module level import not at top of file
./nodes/test/nodes_tests/test_admin_extensions.py:12:1: E402 module level import not at top of file
./nodes/test/nodes_tests/test_admin_extensions.py:14:1: E402 module level import not at top of file
./nodes/test/nodes_tests/test_admin_extensions.py:15:1: E402 module level import not at top of file
./nodes/test/nodes_tests/test_admin_extensions.py:16:1: E402 module level import not at top of file
./nodes/test/nodes_tests/test_admin_extensions.py:17:1: E402 module level import not at top of file
./nodes/test/nodes_tests/test_admin_extensions.py:18:1: E402 module level import not at top of file
./nodes/test/nodes_tests/test_plugins.py:19:1: E402 module level import not at top of file
./nodes/test/nodes_tests/test_plugins.py:21:1: E402 module level import not at top of file
./nodes/test/nodes_tests/test_plugins.py:22:1: E402 module level import not at top of file
./nodes/test/nodes_tests/test_plugins.py:23:1: E402 module level import not at top of file
./nodes/test/nodes_tests/test_plugins.py:25:1: E402 module level import not at top of file
./nodes/test/nodes_tests/test_plugins.py:26:1: E402 module level import not at top of file
./nodes/test/nodes_tests/test_plugins.py:27:1: E402 module level import not at top of file
./nodes/test/nodes_tests/test_plugins.py:28:1: E402 module level import not at top of file
./nodes/test/nodes_tests/test_plugins.py:29:1: E402 module level import not at top of file
./nodes/test/nodes_tests/test_plugins.py:30:1: E402 module level import not at top of file
./nodes/test/nodes_tests/test_plugins.py:31:1: E402 module level import not at top of file
./nodes/test/nodes_tests/test_plugins.py:32:1: E402 module level import not at top of file
./nodes/test/nodes_tests/test_plugins.py:33:1: E402 module level import not at top of file
./nodes/test/nodes_tests/test_plugins.py:34:1: E402 module level import not at top of file
./nodes/test/nodes_tests/test_plugins.py:35:1: E402 module level import not at top of file
./nodes/test/nodes_tests/test_plugins.py:36:1: E402 module level import not at top of file
./nodes/test/nodes_tests/test_plugins.py:37:1: E402 module level import not at top of file
./nodes/test/nodes_tests/test_plugins.py:38:1: E402 module level import not at top of file
./nodes/test/nodes_tests/test_plugins.py:39:1: E402 module level import not at top of file
./nodes/test/nodes_tests/test_plugins.py:40:1: E402 module level import not at top of file
./nodes/test/nodes_tests/test_plugins.py:41:1: E402 module level import not at top of file
./nodes/test/nodes_tests/test_plugins.py:42:1: E402 module level import not at top of file
./nodes/test/nodes_tests/test_plugins.py:43:1: E402 module level import not at top of file
./nodes/test/nodes_tests/test_plugins.py:44:1: E402 module level import not at top of file
./nodes/test/nodes_tests/test_plugins.py:45:1: E402 module level import not at top of file
./nodes/test/nodes_tests/test_plugins.py:46:1: E402 module level import not at top of file
./nodes/test/nodes_tests/test_plugins.py:47:1: E402 module level import not at top of file
./nodes/test/nodes_tests/test_plugins.py:48:1: E402 module level import not at top of file
./server/pulp/server/webservices/wsgi.py:16:1: E402 module level import not at top of file
Associated revisions
Revision 3d60218a
View on GitHub
Fix flake8 errors on build
New flake8 update has some issues. This patch fixes latest issues.
History
#1
Updated by pcreech about 5 years ago
- Status changed from NEW to ASSIGNED
- Assignee set to pcreech
#2
Updated by pcreech about 5 years ago
- Status changed from ASSIGNED to MODIFIED
- % Done changed from 0 to 100
Applied in changeset pulp|3d60218a531dd25c0ca99837f93314aec4c0efec.
#3
Updated by pcreech about 5 years ago
- Status changed from MODIFIED to POST
#4
Updated by pcreech about 5 years ago
- Status changed from POST to MODIFIED
#5
Updated by mhrivnak about 5 years ago
- Triaged changed from No to Yes
#6
Updated by rbarlow almost 5 years ago
- Status changed from MODIFIED to 5
- Platform Release set to 2.8.0
#7
Updated by dkliban@redhat.com almost 5 years ago
- Status changed from 5 to CLOSED - CURRENTRELEASE
#8
Updated by bmbouter almost 2 years ago
- Tags Pulp 2 added
Please register to edit this issue
Fix flake8 errors on build
New flake8 update has some issues. This patch fixes latest issues.
https://pulp.plan.io/issues/1517 closes #1517