use eslint

.eslintrc

@@ -0,0 +1,199 @@
+{
+  "ecmaFeatures": {
+    "jsx": true
+  },
+  "parser": "espree",
+  "env": {
+    "amd": false,
+    "jasmine": false,
+    "node": true,
+    "mocha": true,
+    "browser": true,
+    "builtin": true,
+    "es6": true
+  },
+  "rules": {
+    "no-alert": 2,
+    "no-array-constructor": 2,
+    "no-bitwise": 2,
+    "no-caller": 2,
+    "no-catch-shadow": 2,
+    "no-comma-dangle": 0,
+    "no-cond-assign": [2, "except-parens"],
+    "no-constant-condition": 2,
+    "no-continue": 0,
+    "no-control-regex": 2,
+    "no-debugger": 2,
+    "no-delete-var": 2,
+    "no-div-regex": 0,
+    "no-dupe-keys": 2,
+    "no-dupe-args": 2,
+    "no-duplicate-case": 2,
+    "no-else-return": 0,
+    "no-empty": 2,
+    "no-empty-class": 0,
+    "no-empty-character-class": 2,
+    "no-empty-label": 2,
+    "no-eq-null": 0,
+    "no-eval": 2,
+    "no-ex-assign": 2,
+    "no-extend-native": 2,
+    "no-extra-bind": 2,
+    "no-extra-boolean-cast": 2,
+    "no-extra-parens": 0,
+    "no-extra-semi": 2,
+    "no-fallthrough": 2,
+    "no-floating-decimal": 0,
+    "no-func-assign": 2,
+    "no-implied-eval": 2,
+    "no-inline-comments": 0,
+    "no-inner-declarations": [2, "functions"],
+    "no-invalid-regexp": 2,
+    "no-irregular-whitespace": 2,
+    "no-iterator": 0,
+    "no-label-var": 2,
+    "no-labels": 2,
+    "no-lone-blocks": 2,
+    "no-lonely-if": 0,
+    "no-loop-func": 2,
+    "no-mixed-requires": [0, false],
+    "no-mixed-spaces-and-tabs": [2, false],
+    "linebreak-style": [0, "unix"],
+    "no-multi-spaces": 2,
+    "no-multi-str": 2,
+    "no-multiple-empty-lines": [0, {
+      "max": 2
+    }],
+    "no-native-reassign": 2,
+    "no-negated-in-lhs": 2,
+    "no-nested-ternary": 0,
+    "no-new": 0,
+    "no-new-func": 2,
+    "no-new-object": 2,
+    "no-new-require": 0,
+    "no-new-wrappers": 2,
+    "no-obj-calls": 2,
+    "no-octal": 2,
+    "no-octal-escape": 2,
+    "no-param-reassign": 0,
+    "no-path-concat": 0,
+    "no-plusplus": 0,
+    "no-process-env": 0,
+    "no-process-exit": 0,
+    "no-proto": 2,
+    "no-redeclare": 0,
+    "no-regex-spaces": 2,
+    "no-reserved-keys": 0,
+    "no-restricted-modules": 0,
+    "no-return-assign": 0,
+    "no-script-url": 2,
+    "no-self-compare": 0,
+    "no-sequences": 2,
+    "no-shadow": 0,
+    "no-shadow-restricted-names": 2,
+    "no-space-before-semi": 0,
+    "no-spaced-func": 2,
+    "no-sparse-arrays": 2,
+    "no-sync": 0,
+    "no-ternary": 0,
+    "no-trailing-spaces": 2,
+    "no-this-before-super": 0,
+    "no-throw-literal": 0,
+    "no-undef": 2,
+    "no-undef-init": 2,
+    "no-undefined": 0,
+    "no-unexpected-multiline": 0,
+    "no-underscore-dangle": 0,
+    "no-unneeded-ternary": 0,
+    "no-unreachable": 2,
+    "no-unused-expressions": 0,
+    "no-unused-vars": 2,
+    "no-use-before-define": [2, "nofunc"],
+    "no-void": 0,
+    "no-var": 0,
+    "prefer-const": 0,
+    "no-warning-comments": [0, {
+      "terms": ["todo", "fixme", "xxx"],
+      "location": "start"
+    }],
+    "no-with": 2,
+    "array-bracket-spacing": [0, "never"],
+    "accessor-pairs": 0,
+    "block-scoped-var": 0,
+    "brace-style": [0, "1tbs"],
+    "camelcase": 0,
+    "comma-dangle": 0,
+    "comma-spacing": 2,
+    "comma-style": 0,
+    "complexity": [0, 11],
+    "computed-property-spacing": [0, "never"],
+    "consistent-return": 0,
+    "consistent-this": [0, "that"],
+    "constructor-super": 0,
+    "curly": [2, "all"],
+    "default-case": 0,
+    "dot-location": 0,
+    "dot-notation": 0,
+    "eol-last": 2,
+    "eqeqeq": 2,
+    "func-names": 0,
+    "func-style": [0, "declaration"],
+    "generator-star": 0,
+    "generator-star-spacing": 0,
+    "global-strict": [0, "never"],
+    "guard-for-in": 0,
+    "handle-callback-err": 0,
+    "indent": [2, 2],
+    "key-spacing": [2, {
+      "beforeColon": false,
+      "afterColon": true
+    }],
+    "lines-around-comment": 0,
+    "max-depth": [0, 4],
+    "max-len": [0, 80, 4],
+    "max-nested-callbacks": [0, 2],
+    "max-params": [0, 3],
+    "max-statements": [0, 10],
+    "new-cap": 0,
+    "new-parens": 2,
+    "newline-after-var": 0,
+    "object-curly-spacing": [0, "never"],
+    "object-shorthand": 0,
+    "one-var": 0,
+    "operator-assignment": [0, "always"],
+    "operator-linebreak": 0,
+    "padded-blocks": 0,
+    "quote-props": 0,
+    "quotes": [2, "single"],
+    "radix": 0,
+    "semi": 2,
+    "semi-spacing": [2, {
+      "before": false,
+      "after": true
+    }],
+    "sort-vars": 0,
+    "space-after-function-name": [0, "never"],
+    "space-after-keywords": [0, "always"],
+    "space-before-blocks": [0, "always"],
+    "space-before-function-paren": [0, "always"],
+    "space-before-function-parentheses": [0, "always"],
+    "space-in-brackets": [0, "never"],
+    "space-in-parens": [0, "never"],
+    "space-infix-ops": 2,
+    "space-return-throw-case": 2,
+    "space-unary-ops": [2, {
+      "words": true,
+      "nonwords": false
+    }],
+    "spaced-comment": 0,
+    "spaced-line-comment": [0, "always"],
+    "strict": [2, "global"],
+    "use-isnan": 2,
+    "valid-jsdoc": 0,
+    "valid-typeof": 0,
+    "vars-on-top": 0,
+    "wrap-iife": 0,
+    "wrap-regex": 0,
+    "yoda": [2, "never"]
+  }
+}

index.js

@@ -1,7 +1,7 @@
 /**!
  * default-user-agent - index.js
  *
- * Copyright(c) fengmk2 and other contributors.
+ * Copyright(c) node-modules and other contributors.
  * MIT Licensed
  *
  * Authors:

package.json

@@ -7,23 +7,19 @@
     "index.js"
   ],
   "scripts": {
-    "test": "mocha --check-leaks -R spec -t 5000 test/*.test.js",
-    "test-cov": "istanbul cover node_modules/.bin/_mocha -- --check-leaks -t 5000 test/*.test.js",
-    "test-travis": "istanbul cover node_modules/.bin/_mocha --report lcovonly -- --check-leaks -t 5000 test/*.test.js",
-    "lint": "jshint .",
+    "test": "npm run lint && mocha --check-leaks -R spec -t 5000 test/*.test.js",
+    "test-cov": "npm run lint && istanbul cover node_modules/.bin/_mocha -- --check-leaks -t 5000 test/*.test.js",
+    "test-travis": "npm run lint && istanbul cover node_modules/.bin/_mocha --report lcovonly -- --check-leaks -t 5000 test/*.test.js",
+    "lint": "eslint index.js test",
     "autod": "autod -w --prefix '~'",
-    "cnpm": "npm install --registry=https://registry.npm.taobao.org",
-    "contributors": "contributors -f plain -o AUTHORS"
+    "cnpm": "npm install --registry=https://registry.npm.taobao.org"
   },
   "dependencies": {
     "os-name": "~1.0.3"
   },
   "devDependencies": {
     "autod": "*",
-    "contributors": "*",
-    "should": "*",
-    "jshint": "*",
-    "cov": "*",
+    "eslint": "1",
     "istanbul": "*",
     "mocha": "*"
   },

test/default-user-agent.test.js

@@ -1,36 +1,38 @@
 /**!
  * default-user-agent - test/default-user-agent.test.js
  *
- * Copyright(c) 2014 fengmk2 and other contributors.
+ * Copyright(c) node-modules and other contributors.
  * MIT Licensed
  *
  * Authors:
  *   fengmk2 <fengmk2@gmail.com> (http://fengmk2.com)
  */
 
-"use strict";
+'use strict';
 
 /**
  * Module dependencies.
  */
 
-var should = require('should');
+var assert = require('assert');
 var ua = require('../');
 
 describe('default-user-agent.test.js', function () {
   it('should got default user agent string', function () {
     var s = ua();
-    s.should.be.a.String;
-    s.should.match(/\w+\/\w+/);
+    assert.equal(typeof s, 'string');
+    assert(/\w+\/\w+/.test(s));
+    console.log(s);
     // 'Node.js/0.11.15 (OS X Yosemite; x64)'
-    s.should.match(/^Node\.js\/\d\.\d+\.\d+ \(\w+ .+; \w+\)$/);
+    assert(/^Node\.js\/\d+\.\d+\.\d+ \(\w+ .+; \w+\)$/);
   });
 
   it('should got custom user agent string', function () {
     var s = ua('urllib', '0.1.1');
-    s.should.be.a.String;
-    s.should.match(/\w+\/\w+/);
+    assert.equal(typeof s, 'string');
+    assert(/\w+\/\w+/.test(s));
+    console.log(s);
     // 'Node.js/0.11.15 (OS X Yosemite; x64)'
-    s.should.match(/^urllib\/0\.1\.1 Node\.js\/\d\.\d+\.\d+ \(\w+ .+; \w+\)$/);
+    assert(/^urllib\/0\.1\.1 Node\.js\/\d+\.\d+\.\d+ \(\w+ .+; \w+\)$/.test(s));
   });
 });