update dependencies (#164)

Author: bartveneman

.travis.yml

@@ -1,7 +1,6 @@
 language: node_js
 node_js:
-  - '8'
-  - '9'
   - '10'
+  - '12'
 after_script:
   - 'cat coverage/lcov.info | ./node_modules/.bin/coveralls'

package-lock.json

@@ -36,7 +36,8 @@
     "space": true,
     "semicolon": false,
     "rules": {
-      "operator-linebreak": "off"
+      "operator-linebreak": "off",
+      "space-before-function-paren": "off"
     }
   },
   "prettier": {
@@ -63,24 +64,24 @@
     "css-time-sort": "^1.0.0",
     "css-unit-sort": "^3.3.1",
     "css-value-browser-h4cks-analyzer": "^2.0.0",
-    "flat": "^4.1.0",
+    "flat": "^5.0.0",
     "gzip-size": "^5.1.1",
     "is-vendor-prefixed": "^3.3.1",
     "path": "^0.12.7",
-    "postcss": "^7.0.17",
+    "postcss": "^7.0.26",
     "postcss-values-parser": "^3.0.5",
     "specificity": "^0.4.1",
     "split-css-value": "^0.1.1",
     "split-lines": "^2.0.0",
-    "string-natural-compare": "^2.0.3",
+    "string-natural-compare": "^3.0.0",
     "tinycolor2": "^1.4.1",
     "vendor-prefixes": "^1.0.0"
   },
   "devDependencies": {
-    "ava": "^2.3.0",
-    "coveralls": "^3.0.6",
-    "nyc": "^14.1.1",
-    "xo": "^0.24.0"
+    "ava": "^2.4.0",
+    "coveralls": "^3.0.9",
+    "nyc": "^15.0.0",
+    "xo": "^0.25.3"
   },
   "nyc": {
     "reporter": [

package.json

@@ -17,10 +17,9 @@ module.exports = rules => {
     .filter(({selectors}) => selectors.length > 0)
     .sort((a, b) => {
       if (a.selectors.length === b.selectors.length) {
-        return compareStrings.caseInsensitive(
-          a.selectors.join(''),
-          b.selectors.join('')
-        )
+        return compareStrings(a.selectors.join(''), b.selectors.join(''), {
+          caseInsensitive: true
+        })
       }
 
       return a.selectors.length - b.selectors.length

src/analyzer/rules/index.js

@@ -7,7 +7,7 @@ module.exports = selectors => {
 
     try {
       complexity = selectorComplexity(selector)
-    } catch (error) {
+    } catch (_) {
       // Fail silently, ignoring the error
     }
 

src/analyzer/selectors/complexity.js

@@ -1,10 +1,10 @@
 const specificity = require('specificity')
-const {caseInsensitive: stringCompare} = require('string-natural-compare')
+const stringCompare = require('string-natural-compare')
 
 // Sort by identifiers count (high to low), then by alphabet (A-Z)
 function sortByIdentifiersCount(a, b) {
   if (a.count === b.count) {
-    return stringCompare(a.value, b.value)
+    return stringCompare(a.value, b.value, {caseInsensitive: true})
   }
 
   return b.count - a.count

src/analyzer/selectors/identifiers.js

@@ -19,7 +19,7 @@ function extractColorsFromDeclaration(declaration) {
         return colors.push(node)
       }
     })
-  } catch (error) {}
+  } catch (_) {}
 
   if (colors.length > 0) {
     declaration.colors = colors.map(color => color.toString().trim())

src/analyzer/values/colors.js

@@ -1,7 +1,11 @@
 const stringSortFn = require('string-natural-compare')
 
 module.exports = (values, sortFn) => {
-  sortFn = sortFn || stringSortFn.caseInsensitive
+  sortFn =
+    sortFn ||
+    function(a, b) {
+      return stringSortFn(String(a), String(b), {caseInsensitive: true})
+    }
 
   // Create a Map of unique values and their counts
   const reduced = [

src/utils/uniquer.js

@@ -60,5 +60,5 @@ test('it ignores !importants when looking for unique declarations', t => {
   ]
   const {totalUnique: actual} = analyze(fixture)
 
-  t.deepEqual(actual, 1)
+  t.is(actual, 1)
 })