congress

run_tests.sh pep8 checker should ignore contrib directory

Bug #1418263 reported by Aaron Rosen
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
congress
Fix Released
Low
David Cheperdak
congress liberty-2

Bug Description

Running run_tests.sh the pep8 checker checks files in contrib. tox -epep8 doesn't do this. run_tests.sh needs to be fixed so this doesn't occur

Tim Hinrichs (thinrichs)
Changed in congress:
importance: Undecided → Low
David Cheperdak (djbchepe)
Changed in congress:
assignee: nobody → David Cheperdak (djbchepe)
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/183950

Changed in congress:
status: New → In Progress
Revision history for this message
Masahito Muroi (muroi-masahito) wrote :

tox -e pep8 also checks codes in contrib directory now. It looks like we don't need to skip pep8 checking in run_test.sh.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to congress (master)

Reviewed: https://review.openstack.org/183950
Committed: https://git.openstack.org/cgit/openstack/congress/commit/?id=495f342688de101c9f408644e4b2ce5921f7b319
Submitter: Jenkins
Branch: master

commit 495f342688de101c9f408644e4b2ce5921f7b319
Author: David Cheperdak <email address hidden>
Date: Sun May 17 13:55:45 2015 -0700

    Added support for folder and file exclusion for pep8 analysis

    Change-Id: I32cfe328a3269d801b4baa8fba1d666b17f492ab
    Closes-Bug: #1418263

Changed in congress:
status: In Progress → Fix Committed
Tim Hinrichs (thinrichs)
Changed in congress:
milestone: none → liberty-2
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.