Fix globbing for eslint
authorBrad Jorsch <bjorsch@wikimedia.org>
Tue, 22 Nov 2016 19:34:02 +0000 (14:34 -0500)
committerBrad Jorsch <bjorsch@wikimedia.org>
Tue, 22 Nov 2016 19:34:02 +0000 (14:34 -0500)
Grunt's `**` matches symlinks but doesn't follow them, so `**/*.js`
matches files that `!extensions/**` doesn't see to exclude.

Bug: T151381
Change-Id: I5d8ae6e24f2fa09efc97db871b0c08d4fbc68efd

Gruntfile.js

index b38bc9a..3fee6ba 100644 (file)
@@ -22,14 +22,15 @@ module.exports = function ( grunt ) {
                                '**/*.js',
                                '!docs/**',
                                '!tests/**',
-                               '!extensions/**',
                                '!node_modules/**',
                                '!resources/lib/**',
                                '!resources/src/jquery.tipsy/**',
                                '!resources/src/jquery/jquery.farbtastic.js',
                                '!resources/src/mediawiki.libs/**',
-                               '!skins/**',
                                '!vendor/**',
+                               // Explicitly say "**/*.js" here in case of symlinks
+                               '!extensions/**/*.js',
+                               '!skins/**/*.js',
                                // Skip functions aren't even parseable
                                '!resources/src/dom-level2-skip.js',
                                '!resources/src/es5-skip.js',