Skip to content

Commit 69239ce

Browse files
authored
Cleanup Effort (#1058)
* 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
1 parent e6ccbaf commit 69239ce

File tree

97 files changed

+3041
-2914
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

97 files changed

+3041
-2914
lines changed

.editorconfig

+6-9
Original file line numberDiff line numberDiff line change
@@ -1,16 +1,13 @@
1-
root = true
1+
# editorconfig.org
22

33
[*]
4-
indent_style = tab
5-
indent_size = 4
64
charset = utf-8
7-
trim_trailing_whitespace = true
8-
insert_final_newline = true
9-
max_line_length = 233
10-
11-
[*.json]
125
indent_style = space
136
indent_size = 2
7+
end_of_line = lf
8+
insert_final_newline = true
9+
trim_trailing_whitespace = true
1410

15-
[*.md]
11+
[.md]
12+
insert_final_newline = false
1613
trim_trailing_whitespace = false

.eslintignore

Whitespace-only changes.

.eslintrc

+17-48
Original file line numberDiff line numberDiff line change
@@ -1,50 +1,19 @@
11
{
2-
"root": true,
3-
"extends": [
4-
"eslint:recommended"
5-
],
6-
"env": {
7-
"node": true,
8-
"mocha": true,
9-
"browser": true,
10-
"es6": true
11-
},
12-
"rules": {
13-
"quotes": ["error", "double"],
14-
"no-undef": "error",
15-
"brace-style": "error",
16-
"eol-last": "error",
17-
"indent": ["error", "tab", { "SwitchCase": 1 }],
18-
"no-extra-bind": "warn",
19-
"no-empty": "off",
20-
"no-multiple-empty-lines": "error",
21-
"no-multi-spaces": "error",
22-
"no-process-exit": "warn",
23-
"space-in-parens": "error",
24-
"no-trailing-spaces": "error",
25-
"no-unused-vars": "error",
26-
"key-spacing": "error",
27-
"space-infix-ops": "error",
28-
"no-unsafe-negation": "error",
29-
"no-loop-func": "warn",
30-
"space-before-function-paren": ["error", "never"],
31-
"space-before-blocks": "error",
32-
"object-curly-spacing": ["error", "always"],
33-
"keyword-spacing": ["error", {
34-
"after": false,
35-
"overrides": {
36-
"try": {"after": true},
37-
"else": {"after": true},
38-
"throw": {"after": true},
39-
"case": {"after": true},
40-
"return": {"after": true},
41-
"finally": {"after": true},
42-
"do": {"after": true}
43-
}
44-
}],
45-
"no-console": "off",
46-
"valid-jsdoc": "error",
47-
"no-useless-computed-key": "error",
48-
"prefer-const": "error"
49-
}
2+
"extends": "webpack",
3+
"globals": {
4+
"document": true,
5+
"window": true
6+
},
7+
"parserOptions": {
8+
"sourceType": "script"
9+
},
10+
"rules": {
11+
"comma-dangle": ["error", "never"],
12+
"consistent-return": "off",
13+
"no-param-reassign": "off",
14+
"no-underscore-dangle": "off",
15+
"prefer-destructuring": ["error", {"object": false, "array": false}],
16+
"prefer-rest-params": "off",
17+
"strict": ["error", "safe"]
18+
}
5019
}

.gitignore

+1
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
1+
npm-debug.log
12
node_modules
23
/client/live.bundle.js
34
/client/index.bundle.js

.travis.yml

+11-4
Original file line numberDiff line numberDiff line change
@@ -2,11 +2,18 @@ sudo: false
22
branches:
33
only:
44
- master
5-
language: node_js
5+
6+
language:
7+
node_js
8+
69
node_js:
7-
- "6"
8-
- "4"
9-
script: npm run travis
10+
- '8'
11+
- '6'
12+
- '4'
13+
14+
script:
15+
npm run ci
16+
1017
after_success:
1118
- cat ./coverage/coverage.json | node_modules/codecov.io/bin/codecov.io.js
1219
- rm -rf ./coverage

0 commit comments

Comments
 (0)