Merge pull request #186 from golmansax/master

Brackets on always shouldn't fail if line is empty
SimenB · Aug 22, 2015 · 5b10dbd · 5b10dbd
2 parents a5b5e11 + e9e3b90
commit 5b10dbd
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 0 deletions.
diff --git a/src/checks/brackets.js b/src/checks/brackets.js
@@ -19,6 +19,7 @@ var brackets = function( line ) {
 	// 3 mixin
 	// 4 .selector,
 	if ( this.state.hashOrCSS ||
+		line.trim().length === 0 ||
 		line.indexOf( ' =' ) !== -1 ||
 		parensRe.test( line ) ||
 		commaRe.test( line ) ) {

diff --git a/test/test.js b/test/test.js
@@ -532,6 +532,9 @@ describe('Linter Style Checks: ', function() {
 			assert.equal( undefined, bracketsTest('{foo() + "bar"}') )
 			assert.equal( undefined, bracketsTest('$foo = {') )
 		})
+		it('undefined if empty', function() {
+			assert.equal( undefined, bracketsTest('  ') )
+		})
 	})
 
 	describe('brackets: disallow brackets', function() {