Merge pull request #184 from GitScrum/milestone-0.1.4

Milestone 0.1.4

Author: Scrum

changelog.md

@@ -1,3 +1,14 @@
+<a name="0.1.4"></a>
+## 0.1.4 (2018-01-22)
+
+* 0.1.4 ([c39cf5a](https://github.com/gitscrum/postcss-at-rules-variables/commit/c39cf5a))
+* fix: issue #183 ([9edfd7c](https://github.com/gitscrum/postcss-at-rules-variables/commit/9edfd7c)), closes [#183](https://github.com/gitscrum/postcss-at-rules-variables/issues/183)
+* test: for issue #183 ([7152277](https://github.com/gitscrum/postcss-at-rules-variables/commit/7152277)), closes [#183](https://github.com/gitscrum/postcss-at-rules-variables/issues/183)
+* test: for issue #183 with rule ([1bce741](https://github.com/gitscrum/postcss-at-rules-variables/commit/1bce741)), closes [#183](https://github.com/gitscrum/postcss-at-rules-variables/issues/183)
+* build: update changelog ([6d1c8bf](https://github.com/gitscrum/postcss-at-rules-variables/commit/6d1c8bf))
+
+
+
 <a name="0.1.3"></a>
 ## 0.1.3 (2018-01-17)
 

package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "postcss-at-rules-variables",
-  "version": "0.1.3",
+  "version": "0.1.4",
   "description": "PostCss plugin to use CSS Custom Properties in at-rule @each, @for, @if, @else ",
   "license": "MIT",
   "repository": "gitscrum/postcss-at-rules-variables",

package.json

@@ -16,8 +16,10 @@ function getProperty(nodes) {
             return;
         }
 
-        rule.each(({prop, value}) => {
-            propertys[prop] = value;
+        rule.each(({type, prop, value}) => {
+            if (type === 'decl') {
+                propertys[prop] = value;
+            }
         });
     });
 

src/index.js

@@ -12,6 +12,18 @@ test('it change circular reference', t => {
     t.is(processing(value), expected);
 });
 
+test('it should not throw error if comment exists', t => {
+    const expected = ':root{ --from: 1; /* comment */ }';
+    const value = ':root{ --from: 1; /* comment */ }';
+    t.is(processing(value), expected);
+});
+
+test('it should not throw error if comment exists with rule', t => {
+    const expected = ':root{ --from: 1; /* comment */ } @for $i from 1 to 2';
+    const value = ':root{ --from: 1; /* comment */ } @for $i from var(--from) to 2';
+    t.is(processing(value), expected);
+});
+
 test('it change first properties for @for', t => {
     const expected = ':root{ --from: 1; } @for $i from 1 to 2';
     const value = ':root{ --from: 1; } @for $i from var(--from) to 2';