Adding horizon to jshint global

Bug #1441780 reported by Thai Tran
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Dashboard (Horizon)
Fix Released
High
Thai Tran

Bug Description

Some legacy functionality like horizon.alert is use in new angular work. Since we are doing jshint checks on new angular work, horizon need to be added to global jshint config to suppress the error.

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

Reviewed: https://review.openstack.org/171762
Committed: https://git.openstack.org/cgit/openstack/horizon/commit/?id=1fa042f233c1b7c790032c6631d487c21ca1f73e
Submitter: Jenkins
Branch: master

commit 1fa042f233c1b7c790032c6631d487c21ca1f73e
Author: Thai Tran <email address hidden>
Date: Wed Apr 8 11:04:02 2015 -0700

    Adding horizon to jshint

    Suppressing error by adding horizon to global jshint configuration.

    Change-Id: Ibd7d0262255a567a1e0c9efcc25bd32a8a3f86b4
    Closes-bug: #1441780

Changed in horizon:
status: In Progress → Fix Committed
Revision history for this message
Rob Cresswell (robcresswell-deactivatedaccount) wrote :

Marked as high because it's a blocker on adding more JShint options, which we really need.

Changed in horizon:
importance: Undecided → High
Changed in horizon:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in horizon:
milestone: liberty-1 → 8.0.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.