Need to fix 152 pep8 errors in fuel-main repository

Bug #1342678 reported by Timur Nurlygayanov
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Released
Low
Vadim Rovachev

Bug Description

Steps To Reproduce:
1. Clone fuel-main repository.
2. Run: pep8 ./ > result.txt ; cat -n result.txt ; rm -rf result.txt

Expected Result:
No pep8 errors.

Observed Result:
152 errors.

Revision history for this message
Timur Nurlygayanov (tnurlygayanov) wrote :
Download full text (13.8 KiB)

     1 ./netcheck.py:24:1: E302 expected 2 blank lines, found 1
     2 ./netcheck.py:28:5: E301 expected 1 blank line, found 0
     3 ./netcheck.py:30:5: E301 expected 1 blank line, found 0
     4 ./netcheck.py:32:5: E301 expected 1 blank line, found 0
     5 ./netcheck.py:34:5: E301 expected 1 blank line, found 0
     6 ./netcheck.py:36:5: E301 expected 1 blank line, found 0
     7 ./netcheck.py:43:5: E301 expected 1 blank line, found 0
     8 ./netcheck.py:45:5: E301 expected 1 blank line, found 0
     9 ./netcheck.py:47:5: E301 expected 1 blank line, found 0
    10 ./netcheck.py:49:5: E301 expected 1 blank line, found 0
    11 ./netcheck.py:52:5: E301 expected 1 blank line, found 0
    12 ./netcheck.py:55:5: E301 expected 1 blank line, found 0
    13 ./netcheck.py:57:5: E301 expected 1 blank line, found 0
    14 ./netcheck.py:65:80: E501 line too long (82 > 79 characters)
    15 ./netcheck.py:210:31: W291 trailing whitespace
    16 ./netcheck.py:235:1: E303 too many blank lines (5)
    17 ./netcheck.py:248:1: E302 expected 2 blank lines, found 1
    18 ./netcheck.py:264:37: W291 trailing whitespace
    19 ./netcheck.py:273:37: W291 trailing whitespace
    20 ./netcheck.py:274:5: E301 expected 1 blank line, found 0
    21 ./netcheck.py:295:1: E303 too many blank lines (5)
    22 ./utils/pairwise_testcases.py:48:72: E203 whitespace before ','
    23 ./utils/pairwise_testcases.py:49:74: E203 whitespace before ','
    24 ./utils/pairwise_testcases.py:50:66: E203 whitespace before ','
    25 ./utils/pairwise_testcases.py:51:65: E203 whitespace before ','
    26 ./utils/pairwise_testcases.py:52:74: E203 whitespace before ','
    27 ./utils/pairwise_testcases.py:53:75: E203 whitespace before ','
    28 ./utils/pairwise_testcases.py:54:73: E203 whitespace before ','
    29 ./utils/pairwise_testcases.py:55:80: E501 line too long (107 > 79 characters)
    30 ./utils/pairwise_testcases.py:69:31: E203 whitespace before ','
    31 ./utils/simple_http_daemon.py:25:1: E302 expected 2 blank lines, found 1
    32 ./utils/simple_http_daemon.py:26:80: E501 line too long (101 > 79 characters)
    33 ./utils/simple_http_daemon.py:35:20: E221 multiple spaces before operator
    34 ./utils/simple_http_daemon.py:36:17: E221 multiple spaces before operator
    35 ./utils/simple_http_daemon.py:46:30: E251 unexpected spaces around keyword / parameter equals
    36 ./utils/simple_http_daemon.py:46:32: E251 unexpected spaces around keyword / parameter equals
    37 ./utils/simple_http_daemon.py:47:18: E251 unexpected spaces around keyword / parameter equals
    38 ./utils/simple_http_daemon.py:47:20: E251 unexpected spaces around keyword / parameter equals
    39 ./utils/simple_http_daemon.py:48:20: E251 unexpected spaces around keyword / parameter equals
    40 ./utils/simple_http_daemon.py:48:22: E251 unexpected spaces around keyword / parameter equals
    41 ./utils/simple_http_daemon.py:67:1: W391 blank line at end of file
    42 ./utils/tftp_probe.py:31:1: E302 expected 2 blank lines, found 1
    43 ./utils/tftp_probe.py:34:80: E501 line too long (82 > 79 characters)
    44 ./utils/tftp_probe.py:118:1: W391 blank line a...

tags: added: low-hanging-fruit
Changed in fuel:
assignee: nobody → Vadim Rovachev (vrovachev)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-main (master)

Fix proposed to branch: master
Review: https://review.openstack.org/107406

Changed in fuel:
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-main (master)

Reviewed: https://review.openstack.org/107406
Committed: https://git.openstack.org/cgit/stackforge/fuel-main/commit/?id=0912b7066e4660784fcd27c7d6fc66757df40646
Submitter: Jenkins
Branch: master

commit 0912b7066e4660784fcd27c7d6fc66757df40646
Author: vrovachev <email address hidden>
Date: Wed Jul 16 18:28:01 2014 +0400

    fix all pep8 errors

    fix pep8 errors in all .py files

    Change-Id: Ic8b3da8861176a34e8660071d769ea0f408052ea
    Closes-Bug: #1342678

Changed in fuel:
status: In Progress → Fix Committed
Changed in fuel:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.