[buddypress-trac] [BuddyPress Trac] #6756: npm check-dependencies

[buddypress-trac]

#6756: npm check-dependencies
-----------------------------------+------------------
 Reporter:  DJPaul                 |       Owner:
     Type:  enhancement            |      Status:  new
 Priority:  normal                 |   Milestone:  2.5
Component:  Tools - Build Process  |     Version:
 Severity:  normal                 |  Resolution:
 Keywords:  has-patch              |
-----------------------------------+------------------

Comment (by netweb):

 Replying to [comment:8 hnla]:
 > So grunt-check-dependencies doesn't install it's own dependency 'check-
 dependencies',

 It does, if you apply the [attachment:6756.diff] patch and run `npm
 install` you'll see that `grunt-check-dependencies` installs its NPM
 package to `node_modules/grunt-check-dependencies` '''and''' as part of
 this the dependancy `check-dependencies` for NodeJS v5.x is installed to
 `node_modules/check-dependencies` or for NodeJS < 4.x it will be in
 `node_modules/grunt-check-dependencies/node_modules/check-dependencies`

 > still can't fully puzzle these modules and when/why to use  loadNpmTasks
 or when registerTask; seems odd BP gruntfile has no load tasks and my
 other project gruntfile has both load & register and I configure both with
 same initConfig but register & load respectively to arrive at same
 favourable result - still that's my dilemma to figure out.

 Yes, it does, take a look here:
 https://buddypress.trac.wordpress.org/browser/trunk/Gruntfile.js#L24

 > Installed check-dependencies and updated gruntfile to netwebs patch on
 one of my BP installs and it works great, nicely halts any grunt command
 and would prevent me using outdated modules without realizing, a no
 brainer to go and commit I would have thought!

 It does, you can take a more in depth peek at it by running: `grunt
 checkDependencies --verbose` also

--
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/6756#comment:9>
BuddyPress Trac <http://buddypress.org/>
BuddyPress Trac