Merge pull request #164 from postcss/skip-comments

Skip comments between imports
postcss · Jan 27, 2016 · b109e73 · b109e73
2 parents 8f8f11d + 488dc27
commit b109e73
Show file tree

Hide file tree

Showing 2 changed files with 19 additions and 0 deletions.
diff --git a/lib/parse-statements.js b/lib/parse-statements.js
@@ -70,6 +70,9 @@ function parseMedia(result, atRule) {
 
 function parseImport(result, atRule) {
   var prev = atRule.prev()
+  while (prev && prev.type === "comment") {
+    prev = prev.prev()
+  }
   if (prev) {
     if (
       prev.type !== "atrule" ||

diff --git a/test/lint.js b/test/lint.js
@@ -21,6 +21,22 @@ test("should warn when not @charset and not @import statement before", t => {
   })
 })
 
+test("should not warn if comments before @import", t => {
+  return processor.process(`/* skipped comment */ @import "";`)
+  .then(function(result) {
+    const warnings = result.warnings()
+    t.is(warnings.length, 1)
+    t.is(warnings[0].text, `Unable to find uri in '@import ""'`)
+  })
+})
+
+test("should warn if something before comments", t => {
+  return processor.process(`a{} /* skipped comment */ @import "";`)
+  .then(function(result) {
+    t.is(result.warnings().length, 1)
+  })
+})
+
 test("should not warn when @charset or @import statement before", t => {
   return Promise.all([
     processor.process(`@import "bar.css"; @import "bar.css";`, {