move flake8 option from run_tests.sh to tox.ini

Bug #1214159 reported by Kui Shi
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Identity (keystone)
Fix Released
Low
Morgan Fainberg

Bug Description

In run_tests.sh, the function run_flake8() still keep the flake8 option. They should be moved to the section [flake8] to tox.ini, and keep the function run_flake8() simple.

Kui Shi (skuicloud)
Changed in keystone:
assignee: nobody → Kui Shi (skuicloud)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to keystone (master)

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

Changed in keystone:
status: New → In Progress
Dolph Mathews (dolph)
Changed in keystone:
importance: Undecided → Low
Changed in keystone:
assignee: Kui Shi (skuicloud) → Morgan Fainberg (mdrnstm)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to keystone (master)

Reviewed: https://review.openstack.org/42766
Committed: http://github.com/openstack/keystone/commit/361185c0beb63a94288b6abc748c9163442f2c82
Submitter: Jenkins
Branch: master

commit 361185c0beb63a94288b6abc748c9163442f2c82
Author: Kui Shi <email address hidden>
Date: Tue Aug 20 06:28:04 2013 +0800

    remove flake8 option from run_tests.sh

    flake8 will read the section [flake8] of tox.ini to guide the
    behavior. In run_tests.sh, the function run_flake8() does not
    need the falke8 option any more.

    Fixes Bug #1214159
    Change-Id: I1bbba279a339e35c776a6969598b398f57fd7646

Changed in keystone:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in keystone:
milestone: none → havana-3
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in keystone:
milestone: havana-3 → 2013.2
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.