Opened 10 years ago
Closed 10 years ago
#14504 closed feature (fixed)
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.
Change History (2)
comment:1 Changed 10 years ago by
Component: | unfiled → build |
---|---|
Milestone: | None → 1.11/2.1 |
Owner: | set to m_gol |
Priority: | undecided → low |
Status: | new → assigned |
comment:2 Changed 10 years ago by
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
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
We can't use a single target since we need to pass slightly modified options to
jshint:dist
, but the rest can be done.