Update clients to use latest eslint (#1699)

And fix new errors that were found.

This also adds tools/run-all-js-tests.py which runs `npm install`
and `npm test` in all directories containing a package.json file.
This commit is contained in:
Erik Arvidsson
2016-06-02 09:41:32 -07:00
parent 4ead647a2d
commit d6537c74c5
23 changed files with 59 additions and 23 deletions
+1 -1
View File
@@ -7,7 +7,7 @@
"yargs": "4.4.0"
},
"devDependencies": {
"@attic/eslintrc": "^1.0.0",
"@attic/eslintrc": "^3.1.0",
"babel-cli": "6.6.5",
"babel-core": "6.7.2",
"babel-generator": "6.7.2",
+1 -1
View File
@@ -1,7 +1,7 @@
{
"name": "noms-bar-chart",
"devDependencies": {
"@attic/eslintrc": "^1.0.0",
"@attic/eslintrc": "^3.1.0",
"@attic/noms": "^37.0.0",
"@attic/webpack-config": "^2.1.0",
"babel-cli": "6.6.5",
+1 -1
View File
@@ -8,7 +8,7 @@
"babel-regenerator-runtime": "6.5.0"
},
"devDependencies": {
"@attic/eslintrc": "^1.0.0",
"@attic/eslintrc": "^3.1.0",
"babel-cli": "6.6.5",
"babel-core": "6.7.2",
"babel-generator": "6.7.2",
+1 -1
View File
@@ -10,7 +10,7 @@
"request": "^2.72.0"
},
"devDependencies": {
"@attic/eslintrc": "^2.0.0",
"@attic/eslintrc": "^3.1.0",
"babel-cli": "6.6.5",
"babel-core": "6.7.2",
"babel-generator": "6.7.2",
+1 -1
View File
@@ -7,7 +7,7 @@
import argv from 'yargs';
import request from 'request';
import parse from 'csv-parse';
import {DatasetSpec, List, ListWriter, Struct, newStruct, escapeStructField} from '@attic/noms';
import {DatasetSpec, ListWriter, Struct, newStruct, escapeStructField} from '@attic/noms';
const args = argv
.usage('Usage: $0 <url> <dataset>')
+1 -1
View File
@@ -8,7 +8,7 @@
"yargs": "4.4.0"
},
"devDependencies": {
"@attic/eslintrc": "^1.0.0",
"@attic/eslintrc": "^3.1.0",
"babel-cli": "6.6.5",
"babel-core": "6.7.2",
"babel-generator": "6.7.2",
+1 -1
View File
@@ -9,7 +9,7 @@
"yargs": "4.4.0"
},
"devDependencies": {
"@attic/eslintrc": "^1.0.0",
"@attic/eslintrc": "^3.1.0",
"babel-cli": "6.6.5",
"babel-core": "6.7.2",
"babel-generator": "6.7.2",
+1 -1
View File
@@ -10,7 +10,7 @@
"yargs": "4.4.0"
},
"devDependencies": {
"@attic/eslintrc": "^1.0.0",
"@attic/eslintrc": "^3.1.0",
"babel-cli": "6.6.5",
"babel-core": "6.7.2",
"babel-generator": "6.7.2",
+1 -1
View File
@@ -10,7 +10,7 @@
"yargs": "4.4.0"
},
"devDependencies": {
"@attic/eslintrc": "^1.0.0",
"@attic/eslintrc": "^3.1.0",
"babel-cli": "6.6.5",
"babel-core": "6.7.2",
"babel-generator": "6.7.2",
+1 -1
View File
@@ -10,7 +10,7 @@
"yargs": "4.4.0"
},
"devDependencies": {
"@attic/eslintrc": "^1.0.0",
"@attic/eslintrc": "^3.1.0",
"babel-cli": "6.6.5",
"babel-core": "6.7.2",
"babel-generator": "6.7.2",
+1 -1
View File
@@ -8,7 +8,7 @@
"babel-regenerator-runtime": "6.5.0"
},
"devDependencies": {
"@attic/eslintrc": "^1.0.0",
"@attic/eslintrc": "^3.1.0",
"babel-cli": "6.6.5",
"babel-core": "6.7.2",
"babel-generator": "6.7.2",
+1 -1
View File
@@ -37,7 +37,7 @@ async function main(): Promise<void> {
const input = inSpec.dataset();
const commit = await input.head();
const head = commit && commit.value;
invariant(head, quit(`{args._[0]} does not exist}`));
invariant(head, quit(`${args._[0]} does not exist}`));
const pitchers = new Map();
const inningPs = [];
+1 -1
View File
@@ -1,7 +1,7 @@
{
"name": "noms-splore",
"devDependencies": {
"@attic/eslintrc": "^1.0.0",
"@attic/eslintrc": "^3.1.0",
"@attic/noms": "^37.0.0",
"@attic/webpack-config": "^2.1.0",
"babel-cli": "6.6.5",
+2 -2
View File
@@ -218,11 +218,11 @@ function handleNodeClick(e: MouseEvent, id: string) {
if (id.indexOf('/') > -1) {
if (data.links[id] && data.links[id].length > 0) {
data.nodes[id].isOpen = !Boolean(data.nodes[id].isOpen);
data.nodes[id].isOpen = !data.nodes[id].isOpen;
render();
}
} else {
data.nodes[id].isOpen = !Boolean(data.nodes[id].isOpen);
data.nodes[id].isOpen = !data.nodes[id].isOpen;
if (data.links[id] || !data.nodes[id].isOpen) {
render();
} else {
+1 -1
View File
@@ -8,7 +8,7 @@
"yargs": "4.4.0"
},
"devDependencies": {
"@attic/eslintrc": "^1.0.0",
"@attic/eslintrc": "^3.1.0",
"babel-cli": "6.6.5",
"babel-core": "6.7.2",
"babel-generator": "6.7.2",
+1 -1
View File
@@ -16,7 +16,7 @@
"tingodb": "^0.4.2"
},
"devDependencies": {
"@attic/eslintrc": "^2.0.0",
"@attic/eslintrc": "^3.1.0",
"babel-cli": "6.6.5",
"babel-core": "6.7.2",
"babel-generator": "6.7.2",
+1 -1
View File
@@ -10,7 +10,7 @@ import {SequenceCursor} from './sequence.js';
import {invariant} from './assert.js';
import type {ValueReader, ValueWriter, ValueReadWriter} from './value-store.js';
import {blobType} from './type.js';
import {MetaTuple, newIndexedMetaSequenceChunkFn, newIndexedMetaSequenceBoundaryChecker,} from
import {MetaTuple, newIndexedMetaSequenceChunkFn, newIndexedMetaSequenceBoundaryChecker} from
'./meta-sequence.js';
import BuzHashBoundaryChecker from './buzhash-boundary-checker.js';
import Ref from './ref.js';
+2 -2
View File
@@ -16,9 +16,9 @@ import {compare, equals} from './compare.js';
import {sha1Size} from './hash.js';
import {getHashOfValue} from './get-hash.js';
import {getTypeOfValue, makeMapType, makeUnionType} from './type.js';
import {MetaTuple, newOrderedMetaSequenceBoundaryChecker, newOrderedMetaSequenceChunkFn,} from
import {MetaTuple, newOrderedMetaSequenceBoundaryChecker, newOrderedMetaSequenceChunkFn} from
'./meta-sequence.js';
import {OrderedSequence, OrderedSequenceCursor, OrderedSequenceIterator,} from
import {OrderedSequence, OrderedSequenceCursor, OrderedSequenceIterator} from
'./ordered-sequence.js';
import diff from './ordered-sequence-diff.js';
import {ValueBase} from './value.js';
+1 -1
View File
@@ -8,7 +8,7 @@ import {assert} from 'chai';
import {suite, test} from 'mocha';
import List from './list.js';
import {MetaTuple, newOrderedMetaSequenceChunkFn, newIndexedMetaSequenceChunkFn,} from
import {MetaTuple, newOrderedMetaSequenceChunkFn, newIndexedMetaSequenceChunkFn} from
'./meta-sequence.js';
import Ref from './ref.js';
import Set from './set.js';
+2 -2
View File
@@ -16,9 +16,9 @@ import Collection from './collection.js';
import {compare, equals} from './compare.js';
import {getHashOfValue} from './get-hash.js';
import {invariant} from './assert.js';
import {MetaTuple, newOrderedMetaSequenceBoundaryChecker, newOrderedMetaSequenceChunkFn,} from
import {MetaTuple, newOrderedMetaSequenceBoundaryChecker, newOrderedMetaSequenceChunkFn} from
'./meta-sequence.js';
import {OrderedSequence, OrderedSequenceCursor, OrderedSequenceIterator,} from
import {OrderedSequence, OrderedSequenceCursor, OrderedSequenceIterator} from
'./ordered-sequence.js';
import diff from './ordered-sequence-diff.js';
import {makeSetType, makeUnionType, getTypeOfValue} from './type.js';
+3
View File
@@ -8,5 +8,8 @@
"eslint": "^2.11.1",
"eslint-plugin-flow-vars": "^0.4.0",
"eslint-plugin-react": "^5.1.1"
},
"scripts": {
"test": "exit 0"
}
}
+3
View File
@@ -6,5 +6,8 @@
"dependencies": {
"babel-loader": "^6.2.1",
"webpack": "^1.12.11"
},
"scripts": {
"test": "exit 0"
}
}
+30
View File
@@ -0,0 +1,30 @@
#!/usr/bin/env python
# Copyright 2016 The Noms Authors. All rights reserved.
# Licensed under the Apache License, version 2.0:
# http://www.apache.org/licenses/LICENSE-2.0
# This tool finds all package.json files and runs npm install and npm test in those directories.
import os
import subprocess
from contextlib import contextmanager
@contextmanager
def pushd(path):
currentDir = os.getcwd()
os.chdir(path)
yield
os.chdir(currentDir)
def main():
lsfiles = subprocess.check_output(['git', 'ls-files']).split('\n')
for f in lsfiles:
path, name = os.path.split(f)
if name == 'package.json':
with pushd(path):
subprocess.check_call(['npm', 'install'])
subprocess.check_call(['npm', 'test'])
if __name__ == '__main__':
main()