Skip to:
Content

BuddyPress.org

Changeset 8380


Ignore:
Timestamp:
05/04/2014 06:16:12 PM (8 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:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/Gruntfile.js

    r8356 r8380  
    2828        'bp-templates/bp-legacy/js/*.js',
    2929        'bp-xprofile/admin/js/*.js'
     30    ],
     31
     32    BP_EXCLUDED_JS = [
     33        '!bp-templates/bp-legacy/js/*.js',
     34        '!bp-themes/bp-default/_inc/*.js'
    3035    ];
    3136
     
    4247                expand: true,
    4348                cwd: SOURCE_DIR,
    44                 src: BP_JS,
     49
     50                // Exclude known bad JS from jshint for now; see #5613.
     51                src: BP_JS.concat( BP_EXCLUDED_JS ),
    4552
    4653                /**
Note: See TracChangeset for help on using the changeset viewer.