Skip to content
Permalink
Browse files

Build: Style updates

Fixes #14246
Closes gh-1588
  • Loading branch information
arschmitz committed Aug 21, 2015
1 parent 3188e1c commit 8d4157f4e3fbfc03474c65fd151df06dd5c66412
Showing with 5 additions and 40 deletions.
  1. +5 −40 Gruntfile.js
@@ -55,19 +55,6 @@ var
},
component = grunt.option( "component" ) || "**",

jscsBad = [
"ui/button.js",
"ui/datepicker.js",
"ui/draggable.js",
"ui/droppable.js",
"ui/effect.js",
"ui/mouse.js",
"ui/resizable.js",
"ui/selectable.js",
"ui/slider.js",
"ui/sortable.js"
],

htmllintBad = [
"demos/tabs/ajax/content*.html",
"demos/tooltip/ajax/content*.html",
@@ -150,35 +137,13 @@ grunt.initConfig({

// Remove the requireSpacesInsideParentheses override once everything is fixed
jscs: {
"ui-good": [ "ui/*.js" ].concat( jscsBad.map( function( file ) {
return "!" + file;
} ) ),
"ui-bad": {
options: {
requireSpacesInsideParentheses: null
},
src: jscsBad
},
tests: {
options: {
requireSpacesInsideParentheses: null
},
src: "tests/unit/**/*.js"
},
grunt: {
options: {
requireSpacesInsideParentheses: null
},
src: [ "Gruntfile.js", "build/tasks/*.js" ]
},
demos: {
all: {
options: {

// While the style guide removed onevar upgrading jscs to allow it causes too many
// errors right now
disallowMultipleVarDecl: null
requireCapitalizedComments: null
},
src: "demos/**/*.js"
files: {
src: [ "demos/**/*.js", "build/**/*.js", "tests/**/*.js", "ui/**/*.js" ]
}
}
},
uglify: minify,

1 comment on commit 8d4157f

@jzaefferer

This comment has been minimized.

Copy link
Member

@jzaefferer jzaefferer commented on 8d4157f Aug 24, 2015

This isn't working at all. Just add some lint issue to any file and see it pass anyway. Add config: '.jscsrc', to the options and see it spit out 20k errors.

Please sign in to comment.
You can’t perform that action at this time.