Skip to content

Commit

Permalink
Cleanup Effort (#1058)
Browse files Browse the repository at this point in the history
* reorganzing package.json

* reorganzing travis config

* adding eslint-config-webpack

* add eslint ignore

* linting cleanup

* cleanup test linting

* cleanup example linting

* adding comma dangle, fixing prepublish

* fixing linting warnings

* use strict and es6 fixes for node 4.x
  • Loading branch information
shellscape committed Aug 29, 2017
1 parent e6ccbaf commit 69239ce
Show file tree
Hide file tree
Showing 97 changed files with 3,041 additions and 2,914 deletions.
15 changes: 6 additions & 9 deletions .editorconfig
Original file line number Diff line number Diff line change
@@ -1,16 +1,13 @@
root = true
# editorconfig.org

[*]
indent_style = tab
indent_size = 4
charset = utf-8
trim_trailing_whitespace = true
insert_final_newline = true
max_line_length = 233

[*.json]
indent_style = space
indent_size = 2
end_of_line = lf
insert_final_newline = true
trim_trailing_whitespace = true

[*.md]
[.md]
insert_final_newline = false
trim_trailing_whitespace = false
Empty file added .eslintignore
Empty file.
65 changes: 17 additions & 48 deletions .eslintrc
Original file line number Diff line number Diff line change
@@ -1,50 +1,19 @@
{
"root": true,
"extends": [
"eslint:recommended"
],
"env": {
"node": true,
"mocha": true,
"browser": true,
"es6": true
},
"rules": {
"quotes": ["error", "double"],
"no-undef": "error",
"brace-style": "error",
"eol-last": "error",
"indent": ["error", "tab", { "SwitchCase": 1 }],
"no-extra-bind": "warn",
"no-empty": "off",
"no-multiple-empty-lines": "error",
"no-multi-spaces": "error",
"no-process-exit": "warn",
"space-in-parens": "error",
"no-trailing-spaces": "error",
"no-unused-vars": "error",
"key-spacing": "error",
"space-infix-ops": "error",
"no-unsafe-negation": "error",
"no-loop-func": "warn",
"space-before-function-paren": ["error", "never"],
"space-before-blocks": "error",
"object-curly-spacing": ["error", "always"],
"keyword-spacing": ["error", {
"after": false,
"overrides": {
"try": {"after": true},
"else": {"after": true},
"throw": {"after": true},
"case": {"after": true},
"return": {"after": true},
"finally": {"after": true},
"do": {"after": true}
}
}],
"no-console": "off",
"valid-jsdoc": "error",
"no-useless-computed-key": "error",
"prefer-const": "error"
}
"extends": "webpack",
"globals": {
"document": true,
"window": true
},
"parserOptions": {
"sourceType": "script"
},
"rules": {
"comma-dangle": ["error", "never"],
"consistent-return": "off",
"no-param-reassign": "off",
"no-underscore-dangle": "off",
"prefer-destructuring": ["error", {"object": false, "array": false}],
"prefer-rest-params": "off",
"strict": ["error", "safe"]
}
}
1 change: 1 addition & 0 deletions .gitignore
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
npm-debug.log
node_modules
/client/live.bundle.js
/client/index.bundle.js
Expand Down
15 changes: 11 additions & 4 deletions .travis.yml
Original file line number Diff line number Diff line change
Expand Up @@ -2,11 +2,18 @@ sudo: false
branches:
only:
- master
language: node_js

language:
node_js

node_js:
- "6"
- "4"
script: npm run travis
- '8'
- '6'
- '4'

script:
npm run ci

after_success:
- cat ./coverage/coverage.json | node_modules/codecov.io/bin/codecov.io.js
- rm -rf ./coverage

0 comments on commit 69239ce

Please sign in to comment.