Build: Add variable files to linting and update jscs

Closes gh-105

Author: Rohit Mulange

.jshintrc

@@ -12,6 +12,8 @@
 	"trailing": true,
 	"undef": true,
 	"unused": true,
-
-	"node": true
+	"node": true,
+	"globals": {
+		"define": false
+	}
 }

package.json

@@ -51,7 +51,7 @@
     "grunt-csscomb": "3.0.0",
     "grunt-git-authors": "2.0.0",
     "grunt-html": "1.6.0",
-    "grunt-jscs": "0.6.2",
+    "grunt-jscs": "2.0.0",
     "grunt-sass": "0.17.0",
     "grunt-selenium-server": "0.1.2",
     "grunt-svgmin": "2.0.0",

performance/component.js

@@ -7,7 +7,7 @@ var components = {
 				html = "";
 
 			function render( i ) {
-				framework[ component ].variations[ keys[ i ] ].forEach(function( value ) {
+				framework[ component ].variations[ keys[ i ] ].forEach( function( value ) {
 					current[ keys[ i ] ] = value;
 					if ( i < keys.length - 1 ) {
 						render( i + 1 );
@@ -17,7 +17,7 @@ var components = {
 							html = html + framework[ component ].generator.call( this, current );
 						}
 					}
-				});
+				} );
 			}
 			while ( currentCount < count ) {
 				render( 0 );

scss/variables/chassis.js

@@ -1,13 +1,13 @@
-// This is just boiuler plate dont worry about it
+// This is just boiler plate dont worry about it
 // This just supports loading in any enviroment
-(function ( root, factory ) {
-    if ( typeof define === "function" && define.amd ) {
-        define( [], factory );
-    } else if ( typeof exports === "object" ) {
-        module.exports = factory();
-    } else {
-        root.chassis = factory();
-  }
-}( this, function () {
+( function( root, factory ) {
+	if ( typeof define === "function" && define.amd ) {
+		define( [], factory );
+	} else if ( typeof exports === "object" ) {
+		module.exports = factory();
+	} else {
+		root.chassis = factory();
+	}
+}( this, function() {
 return {};
-} ) );
+} ) );

scss/variables/colors.js

@@ -1,12 +1,13 @@
-( function ( root, factory ) {
-    if ( typeof define === 'function' && define.amd ) {
-        define( [ "./chassis" ], factory );
-    } else if (typeof exports === 'object') {
-        module.exports = factory( require('./chassis') );
-    } else {
-        root.chassis = factory( root.chassis );
-    }
-}(this, function ( chassis ) {
+( function( root, factory ) {
+	if ( typeof define === "function" && define.amd ) {
+		define( [ "./chassis" ], factory );
+	} else if ( typeof exports === "object" ) {
+		module.exports = factory( require( "./chassis" ) );
+	} else {
+		root.chassis = factory( root.chassis );
+	}
+}( this, function( chassis ) {
+
 var colors = {
 	"chassis-gray-dark": {
 		"value": "#383838",
@@ -32,7 +33,7 @@ var colors = {
 		"value": "#fadf51",
 		"name": "Chassis Yellow"
 	}
-}
+};
 chassis.colors = {
 	"background": colors[ "chassis-gray-dark" ],
 	"font": colors[ "chassis-gray-dark" ],
@@ -43,6 +44,6 @@ chassis.colors = {
 		name: "Chassis - Blockquote color",
 		value: "#eee"
 	}
-}
+};
 return chassis;
 } ) );

scss/variables/index.js

@@ -1,10 +1,10 @@
-var fs = require( "fs" );
-var path = require( "path" );
-var varPath = path.join( process.cwd(), "scss/variables" );
-var chassis;
+var fs = require( "fs" ),
+ path = require( "path" ),
+ varPath = path.join( process.cwd(), "scss/variables" ),
+ chassis;
 
 fs.readdirSync( varPath ).forEach( function( file ) {
 	chassis = require( path.join( process.cwd(), "scss/variables" ) + "/" + file );
 } );
 
-module.exports = chassis;
+module.exports = chassis;

scss/variables/template.js

@@ -1,13 +1,14 @@
-// This is just boiuler plate dont worry about it
+// This is just boiler plate dont worry about it
 // This just supports loading in any enviroment
-(function ( root, factory ) {
-    if ( typeof define === "function" && define.amd ) {
-        define( [], factory );
-    } else if ( typeof exports === "object" ) {
-        module.exports = factory();
-    } else {
-        root.chassis = factory();
-  }
-}( this, function () {
+( function( root, factory ) {
+	if ( typeof define === "function" && define.amd ) {
+		define( [], factory );
+	} else if ( typeof exports === "object" ) {
+		module.exports = factory();
+	} else {
+		root.chassis = factory();
+	}
+}( this, function() {
+
 //>> return chasses.moduleName;
 } ) );

scss/variables/typography.js

@@ -1,16 +1,16 @@
-( function ( root, factory ) {
-	if ( typeof define === 'function' && define.amd ) {
+( function( root, factory ) {
+	if ( typeof define === "function" && define.amd ) {
 		define( [ "./chassis" ], factory );
-	} else if (typeof exports === 'object') {
-		module.exports = factory( require('./chassis') );
+	} else if ( typeof exports === "object" ) {
+		module.exports = factory( require( "./chassis" ) );
 	} else {
 		root.chassis = factory( root.chassis );
 	}
-}(this, function ( chassis ) {
+}( this, function( chassis ) {
 
-var color = "#222";
-var fontSize = "20px";
-var lineHeight = 1.5;
+var color = "#222",
+	fontSize = "20px",
+	lineHeight = 1.5;
 
 chassis.typography = {
 	normal: {
@@ -43,15 +43,15 @@ chassis.typography = {
 	},
 	monospace: {
 		name: "Font Family - Monospace",
-		value :"\"courier new\", monospace"
+		value: "\"courier new\", monospace"
 	},
 	serif: {
 		name: "Font Family - Serif",
-		value :"Georgia, \"Times New Roman\", Times, serif"
+		value: "Georgia, \"Times New Roman\", Times, serif"
 	},
 	sans: {
 		name: "Font Family - Sans",
-		value :"Helvetica, Arial, sans-serif"
+		value: "Helvetica, Arial, sans-serif"
 	}
 };
 

tasks/options/jscs.js

@@ -1,3 +1,3 @@
 module.exports = {
-	all: [ "*.js", "performance/**/*.js" ]
-};
+	all: [ "*.js", "performance/**/*.js", "scss/variables/**/*.js" ]
+};

tasks/options/jshint.js

@@ -1,6 +1,6 @@
 module.exports = {
-	files: [ "*.js", "performance/**/*.js" ],
+	files: [ "*.js", "performance/**/*.js", "scss/variables/**/*.js" ],
 	options: {
 		jshintrc: ".jshintrc"
 	}
-};
+};