PEP8 max-complexity should be reduced from 33

Bug #1399523 reported by Akihiro Motoki
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Dashboard (Horizon)
Fix Released
Low
Akihiro Motoki

Bug Description

pep8 max-complexity 33 means we have too too large methods.
It should be reduced at least <20 so that we have healthy sized methods.

Changed in horizon:
assignee: nobody → Akihiro Motoki (amotoki)
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to horizon (master)

Reviewed: https://review.openstack.org/129689
Committed: https://git.openstack.org/cgit/openstack/horizon/commit/?id=3b24bca0749c13b828dad5d07733d7b70ca8d435
Submitter: Jenkins
Branch: master

commit 3b24bca0749c13b828dad5d07733d7b70ca8d435
Author: Akihiro Motoki <email address hidden>
Date: Mon Oct 20 01:46:22 2014 +0900

    Split large methods into submethods to reduce max-complexity

    By this refactoring max-complexity of PEP8 is reduced from 33 to 19.

    Closes-Bug: #1399523
    Change-Id: I7e9e5cefd3e72e322d8b327f495ce7fc863c3678

Changed in horizon:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in horizon:
milestone: none → kilo-1
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in horizon:
milestone: kilo-1 → 2015.1.0
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.