Skip to content

Commit fcab128

Browse files
author
Zac Leids
committed
added eslint to project, set arc to run linter when diffing
Reviewers: benchan, arik Reviewed By: arik Subscribers: ben Differential Revision: https://phabricator.bitgo.com/D6106
1 parent 0ec89ee commit fcab128

File tree

7 files changed

+71
-3
lines changed

7 files changed

+71
-3
lines changed

.arcanist-extensions

Submodule .arcanist-extensions added at bd24636

.arcconfig

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,7 @@
11
{
2-
"phabricator.uri" : "http://phabricator.bitgo.com"
2+
"phabricator.uri" : "https://phabricator.bitgo.com",
3+
4+
"load": [
5+
".arcanist-extensions/eslint_linter"
6+
]
37
}

.arclint

Lines changed: 23 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,23 @@
1+
{
2+
"linters": {
3+
"filename": {
4+
"type": "filename"
5+
},
6+
"generated": {
7+
"type": "generated"
8+
},
9+
"merge-conflict": {
10+
"type": "merge-conflict"
11+
},
12+
"nolint": {
13+
"type": "nolint"
14+
},
15+
"spelling": {
16+
"type": "spelling"
17+
},
18+
"eslint": {
19+
"type": "eslint",
20+
"include": "(\\.js$)"
21+
}
22+
}
23+
}

.eslintignore

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,4 @@
1+
node_modules
2+
.idea
3+
.arcanist-extensions
4+
public

.eslintrc.json

Lines changed: 31 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,31 @@
1+
{
2+
"env": {
3+
"browser": true,
4+
"es6": true,
5+
"node": true,
6+
"mocha": true
7+
},
8+
"extends": "eslint:recommended",
9+
"parserOptions": {
10+
"sourceType": "module",
11+
"ecmaVersion": 6
12+
},
13+
"rules": {
14+
"indent": ["error", 2, {"SwitchCase": 1, "MemberExpression": "off"}],
15+
"linebreak-style": ["error", "unix"],
16+
"semi": ["error", "always"],
17+
"eqeqeq": ["warn", "always"],
18+
"curly": "error",
19+
"no-extra-boolean-cast": "off",
20+
"no-unused-vars": "warn",
21+
"object-curly-spacing": ["error", "always", {"objectsInObjects": true, "arraysInObjects": true}],
22+
"array-bracket-spacing": ["error", "never"],
23+
"require-yield": "off",
24+
"func-call-spacing": ["error", "never"],
25+
"quotes": ["error", "single"],
26+
"key-spacing": ["error", { "beforeColon": false, "afterColon": true, "mode": "strict" }],
27+
"quote-props": ["error", "as-needed"],
28+
"no-console": "off",
29+
"no-empty": "warn"
30+
}
31+
}

.gitmodules

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,3 @@
1+
[submodule ".arcanist-extensions"]
2+
path = .arcanist-extensions
3+
url = https://github.com/tagview/arcanist-extensions.git

package.json

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
{
22
"name": "bitgo",
3-
"version": "3.4.11",
3+
"version": "3.4.12",
44
"description": "BitGo Javascript SDK",
55
"main": "./src/index.js",
66
"keywords": [
@@ -21,6 +21,7 @@
2121
"browserify": "13.1.0",
2222
"coveralls": "~2.13.1",
2323
"envify": "~3.4.0",
24+
"eslint": "~4.3.0",
2425
"istanbul": "0.4.5",
2526
"karma": "~0.13.22",
2627
"karma-chrome-launcher": "~0.1.7",
@@ -60,7 +61,8 @@
6061
"test": "npm run test-node",
6162
"express": "node bin/bitgo-express -b 0.0.0.0",
6263
"test-node": "./node_modules/.bin/istanbul test ./node_modules/.bin/_mocha -- --timeout 20000 --reporter list --recursive test/",
63-
"test-browser": "npm run compile-test && ./node_modules/karma/bin/karma start .karma.conf.js --log-level debug --single-run"
64+
"test-browser": "npm run compile-test && ./node_modules/karma/bin/karma start .karma.conf.js --log-level debug --single-run",
65+
"lint": "./node_modules/.bin/eslint --ext .js ."
6466
},
6567
"dependencies": {
6668
"argparse": "~0.1.16",

0 commit comments

Comments
 (0)