jshint need impovement (Horizon)

Bug #1419625 reported by Shaoquan Chen
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Dashboard (Horizon)
Fix Released
Undecided
Shaoquan Chen

Bug Description

It should

- Include JS files under horizon/static/angular/ folder.
- Add .jshintrc to allow globals defined by jQuery, Angular,
  Angular mock, Jasmine and d3.
- Allow global strict mode in spec files.

Tags: horizon-core
Changed in horizon:
assignee: nobody → Shaoquan Chen (sean-chen2)
tags: added: horizon-core
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/153894
Committed: https://git.openstack.org/cgit/openstack/horizon/commit/?id=46b6a3b047024f350cf36e1ef62dc0e54e488720
Submitter: Jenkins
Branch: master

commit 46b6a3b047024f350cf36e1ef62dc0e54e488720
Author: Shaoquan Chen <email address hidden>
Date: Sun Feb 8 12:59:10 2015 -0800

    Improving jshint

    - Including JS files under horizon/static/angular/ folder.
    - Adding .jshintrc to allow globals defined by jQuery, Angular,
      Angular mock, Jasmine, d3 and jsi18n.
    - Allowing global strict mode in spec files.

    Change-Id: I07359a4b3044cd5f675eef3b7df99398df909f81
    Closes-Bug: #1419625

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