Merge pull request #25 from kni-labs/linting-update

linting tweaks
kni-labs · Feb 29, 2016 · 8f95a09 · 8f95a09
2 parents c5fed94 + 3d404d5
commit 8f95a09
Show file tree

Hide file tree

Showing 7 changed files with 14 additions and 18 deletions.
diff --git a/.hound.yml b/.hound.yml
@@ -2,5 +2,5 @@ javascript:
   enabled: false
 
 scss:
-  config_file: .scss-style.yml
+  config_file: .scss-lint.yml
   ignore_file: test/*, knapsack/_reset.scss
diff --git a/.pages b/.pages
diff --git a/.scss-style.yml → .scss-lint.yml b/.scss-style.yml → .scss-lint.yml
@@ -35,7 +35,7 @@ linters:
     enabled: false
     style: short
   HexNotation:
-    enabled: true
+    enabled: false
     style: uppercase
   HexValidation:
     enabled: true
@@ -69,7 +69,7 @@ linters:
   PlaceholderInExtend:
     enabled: false
   PropertyCount:
-    enabled: true
+    enabled: false
     include_nested: false
     max_properties: 10
   PropertySortOrder:

diff --git a/knapsack/_forms.scss b/knapsack/_forms.scss
@@ -60,7 +60,7 @@
   } @else {
     $border-hover-color: if(luminance($bg-color) < $l-tolerance, $bg-hover-color, rgba(#000, .45));
   }
-  $arrow-color: if(luminance($bg-color) < $l-tolerance, rgba(255,255,255, .9), rgba(#000, .25));
+  $arrow-color: if(luminance($bg-color) < $l-tolerance, rgba(255, 255, 255, .9), rgba(#000, .25));
   $arrow-hover-color: if(luminance($bg-color) < $l-tolerance, #FFF, rgba(#000, .45));
 
   border: 1px solid $border-color;

diff --git a/knapsack/_reset.scss b/knapsack/_reset.scss
@@ -90,14 +90,14 @@
   mark,
   audio,
   video {
-    margin: 0;
-    padding: 0;
     border: 0;
-    font-size: 100%;
     font: inherit;
+    font-size: 100%;
+    margin: 0;
+    padding: 0;
     vertical-align: baseline;
   }
-  /* HTML5 display-role reset for older browsers */
+
   article,
   aside,
   details,
@@ -147,14 +147,11 @@
 /// @example
 ///   @include normalize();
 @mixin normalize {
-  /*! normalize.css v3.0.3 | MIT License | github.com/necolas/normalize.css */
+  // normalize.css v3.0.3 | MIT License | github.com/necolas/normalize.css
   html {
     font-family: sans-serif;
-    /* 1 */
     -ms-text-size-adjust: 100%;
-    /* 2 */
     -webkit-text-size-adjust: 100%;
-    /* 2 */
   }
 
   body {
@@ -205,7 +202,6 @@
 
   abbr[title] {
     border-bottom: none;
-    text-decoration: underline;
     text-decoration: underline dotted;
   }
 

diff --git a/knapsack/_utilities.scss b/knapsack/_utilities.scss
@@ -13,11 +13,11 @@
 
   @if $n >= 0 {
     @for $i from 1 through $n {
-        $result: $result * $x;
+      $result: $result * $x;
     }
   } @else {
-      @for $i from $n to 0 {
-        $result: $result / $x;
+    @for $i from $n to 0 {
+      $result: $result / $x;
       }
     }
   @return $result;

diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "knapsack",
-  "version": "0.4.0",
+  "version": "0.4.1",
   "description": "A handy bag of sass utilities and mixins",
   "main": "knapsack/_index.scss",
   "repository": {