chore: update lint configs (googleapis#257)

JustinBeckwith · web-flow · commit d13f1f3c4a43 · 2018-11-06T13:15:04.000-08:00
diff --git a/package.json b/package.json
@@ -57,12 +57,12 @@
     "cover": "nyc --reporter=lcov mocha test/*.js && nyc report",
     "docs": "jsdoc -c .jsdoc.js",
     "generate-scaffolding": "repo-tools generate all && repo-tools generate lib_samples_readme -l samples/ --config ../.cloud-repo-tools.json",
-    "lint": "eslint src/ samples/ system-test/ test/",
-    "prettier": "prettier --write src/*.js src/*/*.js samples/*.js samples/*/*.js test/*.js test/*/*.js system-test/*.js system-test/*/*.js",
+    "lint": "eslint '**/*.js'",
     "samples-test": "cd samples/ && npm link ../ && npm test && cd ../",
     "system-test": "mocha system-test/*.js smoke-test/*.js --timeout 600000",
     "test-no-cover": "mocha test/*.js",
-    "test": "npm run cover"
+    "test": "npm run cover",
+    "fix": "eslint --fix '**/*.js'"
   },
   "dependencies": {
     "@google-cloud/promisify": "^0.3.0",
diff --git a/samples/.eslintrc.yml b/samples/.eslintrc.yml
@@ -1,5 +1,4 @@
 ---
 rules:
   no-console: off
-  
-
+  node/no-missing-require: off
diff --git a/samples/automl/.eslintrc.yml b/samples/automl/.eslintrc.yml
@@ -1,4 +1,4 @@
 ---
 rules:
   no-console: off
-  
+  node/no-missing-require: off
diff --git a/samples/automl/package.json b/samples/automl/package.json
@@ -1,16 +1,17 @@
 {
-  "name": "automl",
+  "name": "automl-samples",
   "version": "1.0.0",
-  "description": "",
+  "description": "A collection of samples for @google-cloud/automl and @google-cloud/vision.",
+  "private": true,
   "main": "automlVisionDataset.js",
   "scripts": {
     "test": "ava -T 1m --verbose system-test/*.test.js"
   },
   "engines": {
     "node": ">=8"
   },
-  "author": "",
-  "license": "ISC",
+  "author": "Google Inc",
+  "license": "Apache-2.0",
   "dependencies": {
     "@google-cloud/automl": "^0.1.1",
     "@google-cloud/vision": "^0.22.0",
diff --git a/samples/automl/system-test/.eslintrc.yml b/samples/automl/system-test/.eslintrc.yml
@@ -1,5 +1,3 @@
 ---
 rules:
   node/no-unpublished-require: off
-  node/no-unsupported-features: off
-  no-empty: off
diff --git a/samples/package.json b/samples/package.json
@@ -4,15 +4,13 @@
   "private": true,
   "license": "Apache-2.0",
   "author": "Google Inc.",
-  "repository": "googleapis/nodejs-vision",
   "engines": {
     "node": ">=8"
   },
   "scripts": {
     "test": "ava -T 1m --verbose system-test/*.test.js"
   },
   "dependencies": {
-
     "@google-cloud/automl": "^0.1.1",
     "@google-cloud/vision": "^0.22.0",
     "async": "^2.6.1",
diff --git a/samples/productSearch/system-test/.eslintrc.yml b/samples/productSearch/system-test/.eslintrc.yml
@@ -1,5 +1,4 @@
 ---
 rules:
   node/no-unpublished-require: off
-  node/no-unsupported-features: off
   no-empty: off
diff --git a/samples/system-test/.eslintrc.yml b/samples/system-test/.eslintrc.yml
@@ -1,5 +1,4 @@
 ---
 rules:
   node/no-unpublished-require: off
-  node/no-unsupported-features: off
   no-empty: off
diff --git a/smoke-test/.eslintrc.yml b/smoke-test/.eslintrc.yml
@@ -0,0 +1,6 @@
+---
+env:
+  mocha: true
+rules:
+  node/no-unpublished-require: off
+  no-console: off
diff --git a/smoke-test/image_annotator_smoke_test.js b/smoke-test/image_annotator_smoke_test.js
@@ -39,7 +39,8 @@ describe('ImageAnnotatorSmokeTest', () => {
       features: features,
     };
     const requests = [requestsElement];
-    client.batchAnnotateImages({requests: requests})
+    client
+      .batchAnnotateImages({requests: requests})
       .then(responses => {
         const response = responses[0];
         console.log(response);
diff --git a/synth.py b/synth.py
@@ -21,23 +21,18 @@
 
 logging.basicConfig(level=logging.DEBUG)
 
+# Run the gapic generator
 gapic = gcp.GAPICGenerator()
-common_templates = gcp.CommonTemplates()
-
-
 versions = ['v1', 'v1p1beta1', 'v1p2beta1', 'v1p3beta1']
-
 for version in versions:
     library = gapic.node_library('vision', version)
     s.copy(library, excludes=['src/index.js', 'README.md', 'package.json'])
 
-templates = common_templates.node_library(package_name="@google-cloud/vision")
+# Copy common templates
+common_templates = gcp.CommonTemplates()
+templates = common_templates.node_library()
 s.copy(templates)
 
-
-'''
-Node.js specific cleanup
-'''
+# Node.js specific cleanup
 subprocess.run(['npm', 'install'])
-subprocess.run(['npm', 'run', 'prettier'])
-subprocess.run(['npm', 'run', 'lint'])
+subprocess.run(['npm', 'run', 'fix'])
diff --git a/system-test/.eslintrc.yml b/system-test/.eslintrc.yml
@@ -2,6 +2,4 @@
 env:
   mocha: true
 rules:
-  node/no-unpublished-require: off
   no-console: off
-  node/no-deprecated-api: off
diff --git a/test/.eslintrc.yml b/test/.eslintrc.yml
@@ -1,6 +1,3 @@
 ---
 env:
   mocha: true
-rules:
-  node/no-unpublished-require: off
-  node/no-deprecated-api: off

-Original file line number
+Diff line change
@@ @@ -1,5 +1,4 @@ @@
 ---
 rules:
   no-console: off
+-
+-
 +  node/no-missing-require: off