From: Brad Jorsch <bjorsch@wikimedia.org>
Date: Tue, 22 Nov 2016 19:34:02 +0000 (-0500)
Subject: Fix globbing for eslint
X-Git-Tag: 1.31.0-rc.0~4795
X-Git-Url: http://git.cyclocoop.org/%7B%24admin_url%7Dmembres/Category:Foo?a=commitdiff_plain;h=19c294112e021042d402a145f2cd1de7c0e41dcc;p=lhc%2Fweb%2Fwiklou.git

Fix globbing for eslint

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
---

diff --git a/Gruntfile.js b/Gruntfile.js
index b38bc9adfc..3fee6bad51 100644
--- a/Gruntfile.js
+++ b/Gruntfile.js
@@ -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',