Changeset 8380

Timestamp:

05/04/2014 06:16:12 PM (11 years ago)

Author:

djpaul

Message:

Grunt: exclude BP-Default and BP-Legacy from jshint

These JS files contains a lot of legacy JS, requiring a lot of effort
to bring up to standards. It’s better to do that incrementally as part
of template improvements, rather than just to make jshint happy. For
now, let’s exclude these from further jshint checks.

Fixes #5613, see also #5160

File:

• trunk/Gruntfile.js (modified) (2 diffs)

trunk/Gruntfile.js

@@ -28 +28 @@
         'bp-templates/bp-legacy/js/*.js',
         'bp-xprofile/admin/js/*.js'
+    ],
+
+    BP_EXCLUDED_JS = [
+        '!bp-templates/bp-legacy/js/*.js',
+        '!bp-themes/bp-default/_inc/*.js'
     ];
 
@@ -42 +47 @@
                 expand: true,
                 cwd: SOURCE_DIR,
-                src: BP_JS,
+
+                // Exclude known bad JS from jshint for now; see #5613.
+                src: BP_JS.concat( BP_EXCLUDED_JS ),
 
                 /**