Merge pull request #283 from jonathantneal/feature/postcss6

Use PostCSS 6

Author: RyanZim

.travis.yml

@@ -1,4 +1,4 @@
 language: node_js
 node_js:
-  - 'stable'
-  - '0.12'
+  - stable
+  - 4.5

appveyor.yml

@@ -2,8 +2,8 @@
 
 environment:
   matrix:
-    - nodejs_version: 5
-    - nodejs_version: 0.12
+    - nodejs_version: 7
+    - nodejs_version: 4.5
 
 version: "{build}"
 build: off

index.js

@@ -1,6 +1,11 @@
+// builtin tooling
 var path = require("path")
+
+// external tooling
 var assign = require("object-assign")
 var postcss = require("postcss")
+
+// internal tooling
 var joinMedia = require("./lib/join-media")
 var resolveId = require("./lib/resolve-id")
 var loadContent = require("./lib/load-content")

lib/parse-statements.js

@@ -1,4 +1,7 @@
+// external tooling
 var valueParser = require("postcss-value-parser")
+
+// extended tooling
 var stringify = valueParser.stringify
 
 function split(params, start) {

lib/process-content.js

@@ -1,5 +1,10 @@
+// builtin tooling
 var path = require("path")
+
+// external tooling
 var postcss = require("postcss")
+
+// placeholder tooling
 var sugarss
 
 module.exports = function processContent(

lib/resolve-id.js

@@ -1,12 +1,5 @@
+// external tooling
 var resolve = require("resolve")
-var jspmResolve
-// Works around https://github.com/jspm/jspm-cli/pull/1779 is released
-try {
-  jspmResolve = require("pkg-resolve")
-}
-catch (ex) {
-  // pass
-}
 
 var moduleDirectories = [
   "web_modules",
@@ -47,12 +40,6 @@ module.exports = function(id, base, options) {
   .catch(function() {
     return resolveModule(id, resolveOpts)
   })
-  .catch(function() {
-    return jspmResolve.default(id, {
-      basedir: resolveOpts.basedir,
-      extensions : resolveOpts.extensions,
-    })
-  })
   .catch(function() {
     if (paths.indexOf(base) === -1) {
       paths.unshift(base)

package.json

@@ -19,18 +19,19 @@
   ],
   "dependencies": {
     "object-assign": "^4.0.1",
-    "postcss": "^5.0.14",
+    "postcss": "^6.0.1",
     "postcss-value-parser": "^3.2.3",
     "read-cache": "^1.0.0",
     "resolve": "^1.1.7"
   },
   "devDependencies": {
-    "ava": "^0.16.0",
-    "eslint": "^1.10.3",
-    "eslint-config-i-am-meticulous": "^2.0.0",
+    "ava": "^0.19.1",
+    "eslint": "^3.19.0",
+    "eslint-config-i-am-meticulous": "^6.0.1",
+    "eslint-plugin-import": "^2.2.0",
     "npmpub": "^3.0.1",
-    "postcss-scss": "^0.1.3",
-    "sugarss": "^0.2.0"
+    "postcss-scss": "^1.0.0",
+    "sugarss": "^1.0.0"
   },
   "jspm": {
     "name": "postcss-import",

test/custom-load.js

@@ -1,4 +1,7 @@
+// external tooling
 import test from "ava"
+
+// internal tooling
 import compareFixtures from "./helpers/compare-fixtures"
 
 test.serial("should accept content", t => {

test/custom-resolve.js

@@ -1,13 +1,20 @@
+// builtin tooling
+import path from "path"
+
+// external tooling
 import test from "ava"
-import compareFixtures from "./helpers/compare-fixtures"
 import postcss from "postcss"
+
+// internal tooling
+import compareFixtures from "./helpers/compare-fixtures"
+
+// plugin
 import atImport from ".."
-import path from "path"
 
 test.serial("should accept file", t => {
   return compareFixtures(t, "custom-resolve-file", {
     resolve: () => {
-      return path.resolve("fixtures/imports/custom-resolve-1.css")
+      return path.resolve("test/fixtures/imports/custom-resolve-1.css")
     },
   })
 })
@@ -16,7 +23,7 @@ test.serial("should accept promised file", t => {
   return compareFixtures(t, "custom-resolve-file", {
     resolve: () => {
       return Promise.resolve(
-        path.resolve("fixtures/imports/custom-resolve-1.css")
+        path.resolve("test/fixtures/imports/custom-resolve-1.css")
       )
     },
   })
@@ -26,9 +33,9 @@ test.serial("should accept array of files", t => {
   return compareFixtures(t, "custom-resolve-array", {
     resolve: () => {
       return [
-        path.resolve("fixtures/imports/custom-resolve-1.css"),
-        path.resolve("fixtures/imports/custom-resolve-2.css"),
-        path.resolve("fixtures/imports/custom-resolve-1.css"),
+        path.resolve("test/fixtures/imports/custom-resolve-1.css"),
+        path.resolve("test/fixtures/imports/custom-resolve-2.css"),
+        path.resolve("test/fixtures/imports/custom-resolve-1.css"),
       ]
     },
   })
@@ -38,9 +45,9 @@ test.serial("should accept promised array of files", t => {
   return compareFixtures(t, "custom-resolve-array", {
     resolve: () => {
       return Promise.resolve([
-        path.resolve("fixtures/imports/custom-resolve-1.css"),
-        path.resolve("fixtures/imports/custom-resolve-2.css"),
-        path.resolve("fixtures/imports/custom-resolve-1.css"),
+        path.resolve("test/fixtures/imports/custom-resolve-1.css"),
+        path.resolve("test/fixtures/imports/custom-resolve-2.css"),
+        path.resolve("test/fixtures/imports/custom-resolve-1.css"),
       ])
     },
   })
@@ -55,11 +62,11 @@ test(
         return path.replace("foo", "imports/bar")
       },
       load: p => {
-        t.is(p, path.resolve("fixtures/imports", "bar.css"))
+        t.is(p, path.resolve("test/fixtures/imports", "bar.css"))
         return "/* comment */"
       },
     }))
-    .process(`@import "foo.css";`, { from: "fixtures/custom-resolve-file" })
+    .process(`@import "foo.css";`, { from: "test/fixtures/custom-resolve-file" })
     .then(result => {
       t.is(result.css, "/* comment */")
     })

test/custom-syntax-parser.js

@@ -1,6 +1,9 @@
+// external tooling
 import test from "ava"
 import scss from "postcss-scss"
 import sugarss from "sugarss"
+
+// internal tooling
 import compareFixtures from "./helpers/compare-fixtures"
 import compareFixturesExt from "./helpers/compare-fixtures-ext"