[wp-trac] [WordPress Trac] #28464: grunt jshint for plugins and plugin developers

WordPress Trac noreply at wordpress.org
Tue Jun 24 21:47:11 UTC 2014


#28464: grunt jshint for plugins and plugin developers
------------------------------+-----------------------------
 Reporter:  MattyRob          |       Owner:  wonderboymusic
     Type:  enhancement       |      Status:  reopened
 Priority:  normal            |   Milestone:  4.0
Component:  Build/Test Tools  |     Version:  trunk
 Severity:  normal            |  Resolution:
 Keywords:  has-patch commit  |     Focuses:
------------------------------+-----------------------------

Comment (by netweb):

 Replying to [comment:12 MattyRob]:
 > Replying to [comment:11 helen]:
 > > grunt precommit needs to not run this, please.
 >
 > Isn't that covered in the patch submitted by netweb above?
 Yes,
 [https://core.trac.wordpress.org/attachment/ticket/28464/28464.corejs.diff
 28464.corejs.diff]​ excludes `jshint:plugins` from both `grunt precommit`
 and `grunt travis:js` tasks.

 I'm not overly excited about the naming in that patch, it add's a task
 `jshint:corejs` because we already have `jshint:core`, I was thinking of
 switching the names of each of those tasks and there are both back compat
 pro's and con's for either. Happy for any other ideas or methods to
 implement this.

--
Ticket URL: <https://core.trac.wordpress.org/ticket/28464#comment:13>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list