fix: update eslint, enforce semi, apply to test dir (#2336)

Author: jstewmon

.eslintignore

@@ -0,0 +1,2 @@
+dist/*
+test/browser/**/*

.eslintrc

@@ -10,13 +10,15 @@
   "rules": {
     "quotes": [2, "single"],
     "strict": 0,
+    "eol-last": ["error", "always"],
     "curly": 0,
     "no-empty": 0,
     "no-underscore-dangle": 0,
     "new-cap": 0,
     "dot-notation": 0,
     "no-use-before-define": 0,
-    "space-after-keywords": [2, "always"],
+    "semi": [2, "always"],
+    "keyword-spacing": [2, {"before": true, "after": true}],
     "space-before-blocks": [2, "always"],
     "no-trailing-spaces": 2,
     "space-unary-ops": 0

dist-tools/client-creator.js

@@ -105,7 +105,7 @@ ClientCreator.prototype.generateClientFileSource = function generateClientFileSo
     }
     code += 'Object.defineProperty(apiLoader.services[\'' + serviceName +'\'], \'' + version + '\', {\n';
     code += tab + 'get: function get() {\n';
-    code += tab + tab + 'var model = require(\'../apis/' + versionInfo.api + '.json\');\n'
+    code += tab + tab + 'var model = require(\'../apis/' + versionInfo.api + '.json\');\n';
     if (versionInfo.hasOwnProperty('paginators')) {
         code += tab + tab + 'model.paginators = require(\'../apis/' + versionInfo.paginators + '.json\').pagination;\n';
     }
@@ -125,7 +125,7 @@ ClientCreator.prototype.generateClientFileSource = function generateClientFileSo
       code: code,
       path: path.join(clientFolderPath, serviceName + '.js'),
       service: serviceName,
-  }
+  };
 };
 
 ClientCreator.prototype.tabs = function tabs(count) {
@@ -134,7 +134,7 @@ ClientCreator.prototype.tabs = function tabs(count) {
       tab += '  ';
   }
   return tab;
-}
+};
 
 ClientCreator.prototype.generateDefinePropertySource = function generateDefinePropertySource(objName, serviceName, className) {
   var tabs = this.tabs;
@@ -211,4 +211,4 @@ ClientCreator.prototype.writeClientServices = function writeClientServices() {
   fs.writeFileSync(browserClientInfo.path, browserClientInfo.code);
 };
 
-module.exports = ClientCreator;
+module.exports = ClientCreator;

dist-tools/create-all-services.js

@@ -3,4 +3,4 @@ var ClientCreator = require('./client-creator');
 var cc = new ClientCreator();
 
 cc.writeClientServices();
-console.log('Finished updating services.');
+console.log('Finished updating services.');

dist-tools/service-collector.js

@@ -165,4 +165,4 @@ function ServiceCollector(services) {
   return serviceCode;
 }
 
-module.exports = ServiceCollector;
+module.exports = ServiceCollector;

dist-tools/webpack.config.rn-core.js

@@ -29,4 +29,4 @@ module.exports = {
             }
         ]
     }
-}
+};

dist-tools/webpack.config.rn-dep.js

@@ -28,4 +28,4 @@ module.exports = {
             }
         ]
     }
-}
+};

dist-tools/webpack.config.rn.js

@@ -31,4 +31,4 @@ module.exports = {
             }
         ]
     }
-}
+};

lib/browser.js

@@ -16,4 +16,4 @@ if (typeof self !== 'undefined') self.AWS = AWS;
  * DO NOT REMOVE
  * browser builder will strip out this line if services are supplied on the command line.
  */
-require('../clients/browser_default');
+require('../clients/browser_default');

lib/browserCryptoLib.js

@@ -34,4 +34,4 @@ module.exports = exports = {
     createSign: function() {
       throw new Error('createSign is not implemented in the browser');
     }
-  }
+  };