Merge pull request #998 from royaldark/develop

Don't ignore comments in no-empty-rulesets
sasstools · Aug 27, 2017 · 2c4f63e · 2c4f63e
2 parents b5aebda + 84796df
commit 2c4f63e
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 2 deletions.
diff --git a/lib/rules/no-empty-rulesets.js b/lib/rules/no-empty-rulesets.js
@@ -18,7 +18,9 @@ module.exports = {
       else {
         block.traverse(function (item) {
           if (!helpers.isEqual(block, item)) {
-            if (item.type !== 'space') {
+            if (item.type !== 'space' &&
+                item.type !== 'singlelineComment' &&
+                item.type !== 'multilineComment') {
               nonSpaceCount++;
             }
           }

diff --git a/tests/rules/no-empty-rulesets.js b/tests/rules/no-empty-rulesets.js
@@ -12,7 +12,7 @@ describe('no empty rulesets - scss', function () {
     lint.test(file, {
       'no-empty-rulesets': 1
     }, function (data) {
-      lint.assert.equal(3, data.warningCount);
+      lint.assert.equal(4, data.warningCount);
       done();
     });
   });

diff --git a/tests/sass/no-empty-rulesets.scss b/tests/sass/no-empty-rulesets.scss
@@ -9,3 +9,10 @@
 }
 
 .bar{}
+
+.baz {
+	// a comment
+	/* and
+	   yet
+	   another */
+}