Skip to main content

Bug Tracker

Side navigation

#14504 closed feature (fixed)

Opened November 01, 2013 07:32PM UTC

Closed December 18, 2013 02:24PM UTC

Build: Upgrade to grunt-contrib-jshint 0.7.1 and squash subtasks

Reported by: scottgonzalez Owned by: m_gol
Priority: low Milestone: 1.11/2.1
Component: build Version: 1.10.2
Keywords: Cc:
Blocked by: Blocking:
Description

grunt-contrib-jshint 0.7.1 allows the jshintrc option to be set to true to have it read the appropriate config file based on the file being checked. The jshint subtasks should be squashed into a single task and use this new setting for simplicity.

See the equivalent commit in jQuery UI.

Attachments (0)
Change History (2)

Changed November 10, 2013 12:46AM UTC by m_gol comment:1

component: unfiledbuild
milestone: None1.11/2.1
owner: → m_gol
priority: undecidedlow
status: newassigned

We can't use a single target since we need to pass slightly modified options to jshint:dist, but the rest can be done.

Changed December 18, 2013 02:24PM UTC by Michał Gołębiowski comment:2

resolution: → fixed
status: assignedclosed

Build: Upgrade to grunt-contrib-jshint 0.7.1 and squash subtasks

grunt-contrib-jshint 0.7.1 allows the jshintrc option to be set to true to have

it read the appropriate config file based on the file being checked. The only

place where we can’t use it is the check for dist/jquery.js that has the onevar

option removed.

Fixes #14504

Changeset: 7deee6af72bfcf328fa09a28a1e68159cc58513f