congress

pep8 can't report any error

Bug #1398683 reported by Lianhao Lu
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
congress
Fix Released
Undecided
Lianhao Lu
congress kilo-3

Bug Description

https://review.openstack.org/#/c/135882/ tried to enable H233, but it actually disabled all the pep8 tests. The ending comma in the ingore variable makess pep8 don't report any errors.

Lianhao Lu (lianhao-lu)
Changed in congress:
assignee: nobody → Lianhao Lu (lianhao-lu)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to congress (master)

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

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

Reviewed: https://review.openstack.org/138647
Committed: https://git.openstack.org/cgit/stackforge/congress/commit/?id=f8f469a6e76f4c2a7c5e5517e9cc96cbbe4ff2f0
Submitter: Jenkins
Branch: master

commit f8f469a6e76f4c2a7c5e5517e9cc96cbbe4ff2f0
Author: Lianhao Lu <email address hidden>
Date: Wed Dec 3 14:15:37 2014 +0800

    Enabled H233/H402 check

    Enabled the following pep8 checks:
    H233 Python 3.x incompatible use of print operator
    H402 one line docstring needs punctuation

    Closes-Bug: #1398683

    Change-Id: I53da19191ac63b5317e2c2ed705bd945229ddb00

Changed in congress:
status: In Progress → Fix Committed
Tim Hinrichs (thinrichs)
Changed in congress:
milestone: none → kilo-3
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

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