better tooling

ignored generated files
moved stuff into folders
added eslint
added travis
added coveralls

Author: sokra

.eslintrc

@@ -0,0 +1,26 @@
+{
+	"parser": "babel-eslint",
+	"env": {
+		"node": true
+	},
+	"ecmaFeatures": {
+		"arrowFunctions": true,
+		"blockBindings": true,
+		"classes": true,
+		"defaultParams": true,
+		"destructuring": true,
+		"forOf": true,
+		"modules": true,
+		"objectLiteralComputedProperties": true,
+		"objectLiteralShorthandMethods": true,
+		"objectLiteralShorthandProperties": true,
+		"spread": true,
+		"superInFunctions": true,
+		"templateStrings": true,
+		"unicodeCodePointEscapes": true,
+		"jsx": true
+	},
+	"rules": {
+		"quotes": "single"
+	}
+}

.gitignore

@@ -1 +1,4 @@
 node_modules
+coverage
+lib
+test

.travis.yml

@@ -0,0 +1,12 @@
+sudo: false
+language: node_js
+node_js:
+  - "0.10"
+  - "0.12"
+  - "iojs"
+script: npm run travis
+
+before_install:
+  - '[ "${TRAVIS_NODE_VERSION}" != "0.10" ] || npm install -g npm'
+
+after_success: cat ./coverage/lcov.info | node_modules/.bin/coveralls --verbose && rm -rf ./coverage

index.js

@@ -2,35 +2,49 @@
   "name": "postcss-modules-extract-imports",
   "version": "0.0.1",
   "description": "A CSS Modules transform to extract local aliases for inline imports",
-  "main": "index.js",
+  "main": "lib/index.js",
   "scripts": {
-    "watch": "chokidar index.src.js -c 'npm run build'",
-    "build": "babel -o index.js index.src.js",
-    "test": "mocha --compilers js:babel/register",
-    "autotest": "chokidar index.src.js test.js -c 'npm test'"
+    "lint": "eslint src",
+    "build": "babel --out-dir lib src",
+    "watch": "chokidar src -c 'npm run build'",
+    "build-test": "babel --out-dir test test-src",
+    "pretest": "npm run lint && npm run build && npm run build-test",
+    "test": "mocha",
+    "autotest": "chokidar src test-src -c 'npm test'",
+    "precover": "npm run lint && npm run build && npm run build-test",
+    "cover": "istanbul cover node_modules/mocha/bin/_mocha",
+    "travis": "npm run cover -- --report lcovonly",
+    "prepublish": "npm run build"
   },
   "repository": {
     "type": "git",
-    "url": "https://github.com/geelen/postcss-modules-extract-imports.git"
+    "url": "https://github.com/css-modules/postcss-modules-extract-imports.git"
   },
   "keywords": [
     "css-modules",
     "postcss",
     "plugin"
   ],
+  "files": [
+    "lib"
+  ],
   "author": "Glen Maddern",
   "license": "ISC",
   "bugs": {
-    "url": "https://github.com/geelen/postcss-modules-extract-imports/issues"
+    "url": "https://github.com/css-modules/postcss-modules-extract-imports/issues"
   },
-  "homepage": "https://github.com/geelen/postcss-modules-extract-imports",
+  "homepage": "https://github.com/css-modules/postcss-modules-extract-imports",
   "dependencies": {
     "postcss": "^4.1.11"
   },
   "devDependencies": {
     "babel": "^5.4.7",
+    "babel-eslint": "^3.1.9",
     "babelify": "^6.1.2",
     "chokidar-cli": "^0.2.1",
+    "coveralls": "^2.11.2",
+    "eslint": "^0.21.2",
+    "istanbul": "^0.3.14",
     "mocha": "^2.2.5"
   }
 }

package.json

@@ -1,28 +1,28 @@
-import postcss from 'postcss'
+import postcss from 'postcss';
 
 const declWhitelist = ['extends'],
   declFilter = new RegExp(`^(${declWhitelist.join('|')})$`),
-  matchImports = /^([\w ]+) from ("([^"]+)"|'([^']+)')$/
+  matchImports = /^([\w ]+) from (?:"([^"]+)"|'([^']+)')$/;
 
-const processor = (css, result) => {
-  let imports = {}
+const processor = (css) => {
+  let imports = {};
 
   // Find any declaration that supports imports
   css.eachDecl(declFilter, (decl) => {
-    let matches = decl.value.match(matchImports)
+    let matches = decl.value.match(matchImports);
     if (matches) {
-      let [_, symbols, _, doubleQuotePath, singleQuotePath] = matches,
-        path = doubleQuotePath || singleQuotePath
-      imports[path] = imports[path] || {}
+      let [/*match*/, symbols, doubleQuotePath, singleQuotePath] = matches;
+      let path = doubleQuotePath || singleQuotePath;
+      imports[path] = imports[path] || {};
       let tmpSymbols = symbols.split(' ')
-        .map(s => imports[path][s] = `__tmp_${s}_${processor.getRandomStr()}`)
-      decl.value = tmpSymbols.join(' ')
+        .map(s => imports[path][s] = `__tmp_${s}_${processor.getRandomStr()}`);
+      decl.value = tmpSymbols.join(' ');
     }
-  })
+  });
 
   // If we've found any imports, insert :import rules
   Object.keys(imports).forEach(path => {
-    let pathImports = imports[path]
+    let pathImports = imports[path];
     css.prepend(postcss.rule({
       selector: `:import("${path}")`,
       before: "\n",
@@ -31,11 +31,11 @@ const processor = (css, result) => {
         value: pathImports[importedSymbol],
         before: "\n  "
       }))
-  }))
-  })
-}
+    }));
+  });
+};
 
-processor.defaultRandomStr = () => Math.random().toString(36).substr(2, 8)
-processor.getRandomStr = processor.defaultRandomStr // Easy to be mocked out
+processor.defaultRandomStr = () => Math.random().toString(36).substr(2, 8);
+processor.getRandomStr = processor.defaultRandomStr; // Easy to be mocked out
 
-export default processor
+export default processor;

index.src.js src/index.jsindex.src.js renamed to src/index.js

@@ -1,6 +1,6 @@
 import assert from "assert"
 import postcss from "postcss"
-import processor from "./index.src.js"
+import processor from "../"
 
 let pipeline = postcss([processor]),
   check = (desc, input, expected, randomStrs) => {