OpenStack Dashboard (Horizon)

Enable H201: Do not write "except:", use "except Exception:" at the very least

Bug #1211657 reported by Kieran Spear
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Dashboard (Horizon)
Fix Released
Low
Kieran Spear
OpenStack Dashboard (Horizon) 2013.2 "havana"

Bug Description

We should gate on H201 as per HACKING.rst

Kieran Spear (kspear)
Changed in horizon:
assignee: nobody → Kieran Spear (kspear)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to horizon (master)

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

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

Reviewed: https://review.openstack.org/41611
Committed: http://github.com/openstack/horizon/commit/242c8df495d8a22bd58797691989852d1b714824
Submitter: Jenkins
Branch: master

commit 242c8df495d8a22bd58797691989852d1b714824
Author: Kieran Spear <email address hidden>
Date: Tue Aug 13 16:54:59 2013 +1000

    Enable H201: do not write "except:"

    Fixes all occurrences of this. We have a custom exception handler
    in Horizon anyway that only catches ClientException in most of these
    cases, but this commit lets us gate on the other cases.

    Change-Id: Iea3dc13817f3e5b775b2024424bf3a906da5584b
    Closes-Bug: #1211657

Changed in horizon:
status: In Progress → Fix Committed
David Lyle (david-lyle)
Changed in horizon:
importance: Undecided → Low
milestone: none → havana-3
Thierry Carrez (ttx)
Changed in horizon:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in horizon:
milestone: havana-3 → 2013.2
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.