Bug Tracker

Ticket #14504 (closed feature: fixed)

Opened 14 months ago

Last modified 12 months ago

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

Reported by: scott.gonzalez Owned by: m_gol
Priority: low Milestone: 1.11/2.1
Component: build Version: 1.10.2
Keywords: Cc:
Blocking: Blocked by:

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.

Change History

comment:1 Changed 13 months ago by m_gol

  • Owner set to m_gol
  • Priority changed from undecided to low
  • Status changed from new to assigned
  • Component changed from unfiled to build
  • Milestone changed from None to 1.11/2.1

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

comment:2 Changed 12 months ago by Michał Gołębiowski

  • Status changed from assigned to closed
  • Resolution set to fixed

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

Note: See TracTickets for help on using tickets.