chore: remove TSLint

On every build, we are basically compiling our code 3 times: once with
jsii, once with eslint and once with tslint. Also, you need 2 plugins
installed into your IDE to get linter erroring and fixing as you type.

Between these, TSLint is deprecated, so why are we even doing this
anymore? Reduce build times by removing TSLint and migrating all the
rules we were getting from it to ESLint.

Make all packages lint correctly, which was also not the case.

packages/@aws-cdk/assert/lib/assertions/have-resource.ts

@@ -101,7 +101,7 @@ export class HaveResourceAssertion extends JestFriendlyAssertion<StackInspector>
   }
 
   public get description(): string {
-    // tslint:disable-next-line:max-line-length
+    // eslint-disable-next-line max-len
     return `resource '${this.resourceType}' with ${JSON.stringify(this.matcher, undefined, 2)}`;
   }
 }

packages/@aws-cdk/assert/lib/synth-utils.ts

@@ -1,7 +1,7 @@
-import * as core from '@aws-cdk/core';
-import * as cxapi from '@aws-cdk/cx-api';
 import * as fs from 'fs';
 import * as path from 'path';
+import * as core from '@aws-cdk/core';
+import * as cxapi from '@aws-cdk/cx-api';
 
 export class SynthUtils {
   public static synthesize(stack: core.Stack, options: core.SynthesisOptions = { }): cxapi.CloudFormationStackArtifact {

packages/@aws-cdk/assert/test/have-output.test.ts

@@ -1,7 +1,7 @@
-import * as cxschema from '@aws-cdk/cloud-assembly-schema';
-import * as cxapi from '@aws-cdk/cx-api';
 import { unlink, writeFileSync } from 'fs';
 import { join } from 'path';
+import * as cxschema from '@aws-cdk/cloud-assembly-schema';
+import * as cxapi from '@aws-cdk/cx-api';
 import '../jest';
 
 let templateFilePath: string;

packages/@aws-cdk/assert/test/have-resource.test.ts

@@ -1,7 +1,7 @@
-import * as cxschema from '@aws-cdk/cloud-assembly-schema';
-import * as cxapi from '@aws-cdk/cx-api';
 import { writeFileSync } from 'fs';
 import { join } from 'path';
+import * as cxschema from '@aws-cdk/cloud-assembly-schema';
+import * as cxapi from '@aws-cdk/cx-api';
 import { ABSENT, arrayWith, exactValue, expect as cdkExpect, haveResource, haveResourceLike } from '../lib/index';
 
 test('support resource with no properties', () => {

packages/@aws-cdk/aws-amplify/lib/app.ts

@@ -210,7 +210,7 @@ export class App extends Resource implements IApp, iam.IGrantable {
         buildSpec: props.autoBranchCreation.buildSpec && props.autoBranchCreation.buildSpec.toBuildSpec(),
         enableAutoBranchCreation: true,
         enableAutoBuild: props.autoBranchCreation.autoBuild === undefined ? true : props.autoBranchCreation.autoBuild,
-        environmentVariables: Lazy.anyValue({ produce: () => renderEnvironmentVariables(this.autoBranchEnvironmentVariables )}, { omitEmptyArray: true }), // tslint:disable-line max-line-length
+        environmentVariables: Lazy.anyValue({ produce: () => renderEnvironmentVariables(this.autoBranchEnvironmentVariables )}, { omitEmptyArray: true }), // eslint-disable-line max-len
         enablePullRequestPreview: props.autoBranchCreation.pullRequestPreview === undefined ? true : props.autoBranchCreation.pullRequestPreview,
         pullRequestEnvironmentName: props.autoBranchCreation.pullRequestEnvironmentName,
         stage: props.autoBranchCreation.stage,

packages/@aws-cdk/aws-apigateway/lib/apigatewayv2.ts

@@ -1,7 +1,7 @@
 // Copyright 2012-2020 Amazon.com, Inc. or its affiliates. All Rights Reserved.
 // Originally generated from the AWS CloudFormation Resource Specification. Now, hand managed.
 
-// tslint:disable:max-line-length
+/* eslint-disable max-len */
 
 import * as cdk from '@aws-cdk/core';
 

packages/@aws-cdk/aws-apigateway/lib/method.ts

@@ -270,7 +270,7 @@ export class Method extends Resource {
       credentials = options.credentialsRole.roleArn;
     } else if (options.credentialsPassthrough) {
       // arn:aws:iam::*:user/*
-      // tslint:disable-next-line:max-line-length
+      // eslint-disable-next-line max-len
       credentials = Stack.of(this).formatArn({ service: 'iam', region: '', account: '*', resource: 'user', sep: '/', resourceName: '*' });
     }
 

packages/@aws-cdk/aws-apigateway/test/authorizers/integ.request-authorizer.handler/index.ts

@@ -1,4 +1,4 @@
-// tslint:disable:no-console
+/* eslint-disable no-console */
 
 export const handler = async (event: any, _context: any = {}): Promise<any> => {
   const authToken: string = event.headers.Authorization;

packages/@aws-cdk/aws-apigateway/test/authorizers/integ.token-authorizer.handler/index.ts

@@ -1,4 +1,4 @@
-// tslint:disable:no-console
+/* eslint-disable no-console */
 
 export const handler = async (event: any, _context: any = {}): Promise<any> => {
   const authToken: string = event.authorizationToken;

packages/@aws-cdk/aws-apigateway/test/integ.cors.handler/index.ts

@@ -1,5 +1,5 @@
 exports.handler = async (evt: any) => {
-  // tslint:disable-next-line:no-console
+  // eslint-disable-next-line no-console
   console.error(JSON.stringify(evt, undefined, 2));
   return {
     statusCode: 200,

packages/@aws-cdk/aws-apigateway/test/test.domains.ts

@@ -1,10 +1,11 @@
-// tslint:disable:object-literal-key-quotes
 import { ABSENT, expect, haveResource } from '@aws-cdk/assert';
 import * as acm from '@aws-cdk/aws-certificatemanager';
 import { Stack } from '@aws-cdk/core';
 import { Test } from 'nodeunit';
 import * as apigw from '../lib';
 
+/* eslint-disable quote-props */
+
 export = {
   'can define either an EDGE or REGIONAL domain name'(test: Test) {
     // GIVEN
@@ -365,7 +366,7 @@ export = {
     });
 
     const testStage = new apigw.Stage(stack, 'test-stage', {
-      deployment : testDeploy,
+      deployment: testDeploy,
     });
 
     // WHEN

packages/@aws-cdk/aws-apigateway/test/test.lambda-api.ts

@@ -4,7 +4,7 @@ import * as cdk from '@aws-cdk/core';
 import { Test } from 'nodeunit';
 import * as apigw from '../lib';
 
-// tslint:disable:object-literal-key-quotes
+/* eslint-disable quote-props */
 
 export = {
   'LambdaRestApi defines a REST API with Lambda proxy integration'(test: Test) {

packages/@aws-cdk/aws-apigateway/test/test.resource.ts

@@ -3,7 +3,7 @@ import { Stack } from '@aws-cdk/core';
 import { Test } from 'nodeunit';
 import * as apigw from '../lib';
 
-// tslint:disable:object-literal-key-quotes
+/* eslint-disable quote-props */
 
 export = {
   'ProxyResource defines a "{proxy+}" resource with ANY method'(test: Test) {

packages/@aws-cdk/aws-apigateway/test/test.restapi.ts

@@ -4,7 +4,7 @@ import { App, CfnElement, CfnResource, Stack } from '@aws-cdk/core';
 import { Test } from 'nodeunit';
 import * as apigw from '../lib';
 
-// tslint:disable:max-line-length
+/* eslint-disable max-len */
 
 export = {
   'minimal setup'(test: Test) {
@@ -956,7 +956,7 @@ export = {
     test.done();
   },
 
-  'Import': {
+  Import: {
     'fromRestApiId()'(test: Test) {
       // GIVEN
       const stack = new Stack();
@@ -995,7 +995,7 @@ export = {
     },
   },
 
-  'SpecRestApi': {
+  SpecRestApi: {
     'add Methods and Resources'(test: Test) {
       // GIVEN
       const stack = new Stack();

packages/@aws-cdk/aws-applicationautoscaling/test/test.step-scaling-policy.ts

@@ -74,7 +74,7 @@ export = {
         return steps.every(step => {
           return reportFalse(intervals.find(interval => {
             const acceptableLowerBounds = step.MetricIntervalLowerBound === -Infinity ? [undefined, 0] : [undefined, step.MetricIntervalLowerBound];
-            // tslint:disable-next-line:max-line-length
+            // eslint-disable-next-line max-len
             const acceptableUpperBounds = step.MetricIntervalUpperBound === Infinity ? [undefined, Infinity] : [undefined, step.MetricIntervalUpperBound];
 
             return (acceptableLowerBounds.includes(interval.lower) && acceptableUpperBounds.includes(interval.upper));
@@ -259,7 +259,7 @@ function apply<T, U>(x: T | undefined, f: (x: T) => U | undefined): U | undefine
  */
 function reportFalse(cond: boolean, ...repr: any[]) {
   if (!cond) {
-    // tslint:disable-next-line:no-console
+    // eslint-disable-next-line no-console
     console.error('PROPERTY FAILS ON:', ...repr);
   }
   return cond;

packages/@aws-cdk/aws-autoscaling/test/auto-scaling-group.test.ts

@@ -8,7 +8,7 @@ import * as cdk from '@aws-cdk/core';
 import { nodeunitShim, Test } from 'nodeunit-shim';
 import * as autoscaling from '../lib';
 
-// tslint:disable:object-literal-key-quotes
+/* eslint-disable quote-props */
 
 nodeunitShim({
   'default fleet'(test: Test) {
@@ -1134,17 +1134,17 @@ nodeunitShim({
 
     // THEN
     expect(stack).to(haveResource('AWS::AutoScaling::AutoScalingGroup', {
-      NotificationConfigurations : [
+      NotificationConfigurations: [
         {
-          TopicARN : { Ref : 'MyTopic86869434' },
-          NotificationTypes : [
+          TopicARN: { Ref: 'MyTopic86869434' },
+          NotificationTypes: [
             'autoscaling:EC2_INSTANCE_LAUNCH_ERROR',
             'autoscaling:EC2_INSTANCE_TERMINATE_ERROR',
           ],
         },
         {
-          TopicARN : { Ref : 'MyTopic86869434' },
-          NotificationTypes : [
+          TopicARN: { Ref: 'MyTopic86869434' },
+          NotificationTypes: [
             'autoscaling:EC2_INSTANCE_TERMINATE',
           ],
         },
@@ -1174,10 +1174,10 @@ nodeunitShim({
 
     // THEN
     expect(stack).to(haveResource('AWS::AutoScaling::AutoScalingGroup', {
-      NotificationConfigurations : [
+      NotificationConfigurations: [
         {
-          TopicARN : { Ref : 'MyTopic86869434' },
-          NotificationTypes : [
+          TopicARN: { Ref: 'MyTopic86869434' },
+          NotificationTypes: [
             'autoscaling:EC2_INSTANCE_LAUNCH',
             'autoscaling:EC2_INSTANCE_LAUNCH_ERROR',
             'autoscaling:EC2_INSTANCE_TERMINATE',
@@ -1206,10 +1206,10 @@ nodeunitShim({
 
     // THEN
     expect(stack).to(haveResource('AWS::AutoScaling::AutoScalingGroup', {
-      NotificationConfigurations : [
+      NotificationConfigurations: [
         {
-          TopicARN : { Ref : 'MyTopic86869434' },
-          NotificationTypes : [
+          TopicARN: { Ref: 'MyTopic86869434' },
+          NotificationTypes: [
             'autoscaling:EC2_INSTANCE_LAUNCH',
             'autoscaling:EC2_INSTANCE_LAUNCH_ERROR',
             'autoscaling:EC2_INSTANCE_TERMINATE',

packages/@aws-cdk/aws-batch/lib/job-definition-image-config.ts

@@ -38,7 +38,6 @@ class TaskDefinition {
    *
    * @internal
    */
-  // tslint:disable-next-line: no-empty
   public _linkContainer() {}
 
   /**

packages/@aws-cdk/aws-batch/test/compute-environment.test.ts

@@ -168,7 +168,7 @@ describe('Batch Compute Evironment', () => {
             generation: ec2.AmazonLinuxGeneration.AMAZON_LINUX_2,
             hardwareType: ecs.AmiHardwareType.STANDARD,
           }),
-          instanceRole:  new iam.CfnInstanceProfile(stack, 'Instance-Profile', {
+          instanceRole: new iam.CfnInstanceProfile(stack, 'Instance-Profile', {
             roles: [ new iam.Role(stack, 'Ecs-Instance-Role', {
               assumedBy: new iam.ServicePrincipal('ec2.amazonaws.com'),
               managedPolicies: [

packages/@aws-cdk/aws-certificatemanager/lambda-packages/dns_validated_certificate_handler/package.json

@@ -10,7 +10,7 @@
   "scripts": {
     "build": "echo No build",
     "test": "jest",
-    "lint": "eslint lib",
+    "eslint": "eslint lib",
     "build+test+package": "npm run build+test",
     "build+test": "npm run build && npm test"
   },

packages/@aws-cdk/aws-certificatemanager/lib/public-suffixes.ts

@@ -1,5 +1,5 @@
 // This file has been generated using ../suffixes/build-map.py
-// tslint:disable:no-trailing-whitespace object-literal-key-quotes
+/* eslint-disable quote-props */
 export const publicSuffixes = {
   'ac': {
     'com': {},

packages/@aws-cdk/aws-cloudformation/test/asset-directory-fixture/index.ts

@@ -1,4 +1,4 @@
-// tslint:disable:no-console
+/* eslint-disable no-console */
 
 exports.handler = async (evt: any) => {
   console.error(JSON.stringify(evt, undefined, 2));

packages/@aws-cdk/aws-cloudformation/test/core-custom-resource-provider-fixture/index.ts

@@ -1,4 +1,4 @@
-// tslint:disable: no-console
+/* eslint-disable no-console */
 
 export function handler(event: any) {
   console.log('I am a custom resource');

packages/@aws-cdk/aws-cloudformation/test/test.nested-stack.ts

@@ -1,13 +1,13 @@
+import * as fs from 'fs';
+import * as path from 'path';
 import { expect, haveResource, matchTemplate, SynthUtils } from '@aws-cdk/assert';
 import * as s3_assets from '@aws-cdk/aws-s3-assets';
 import * as sns from '@aws-cdk/aws-sns';
 import { App, CfnParameter, CfnResource, Construct, ContextProvider, Stack } from '@aws-cdk/core';
-import * as fs from 'fs';
 import { Test } from 'nodeunit';
-import * as path from 'path';
 import { NestedStack } from '../lib/nested-stack';
 
-// tslint:disable:max-line-length
+/* eslint-disable max-len */
 
 export = {
   'fails if defined as a root'(test: Test) {

packages/@aws-cdk/aws-cloudformation/test/test.resource.ts

@@ -5,7 +5,7 @@ import * as cdk from '@aws-cdk/core';
 import { Test, testCase } from 'nodeunit';
 import { CustomResource, CustomResourceProvider } from '../lib';
 
-// tslint:disable:object-literal-key-quotes
+/* eslint-disable quote-props */
 
 export = testCase({
   'custom resources honor removalPolicy': {

packages/@aws-cdk/aws-cloudfront/lib/web_distribution.ts

@@ -517,6 +517,7 @@ export class GeoRestriction {
     }
     locations.forEach(location => {
       if (!GeoRestriction.LOCATION_REGEX.test(location)) {
+        // eslint-disable-next-line max-len
         throw new Error(`Invalid location format for location: ${location}, location should be two-letter and uppercase country ISO 3166-1-alpha-2 code`);
       }
     });
@@ -723,7 +724,7 @@ export class CloudFrontWebDistribution extends cdk.Construct implements IDistrib
       httpVersion: props.httpVersion || HttpVersion.HTTP2,
       priceClass: props.priceClass || PriceClass.PRICE_CLASS_100,
       ipv6Enabled: (props.enableIpV6 !== undefined) ? props.enableIpV6 : true,
-      // tslint:disable-next-line:max-line-length
+      // eslint-disable-next-line max-len
       customErrorResponses: props.errorConfigurations, // TODO: validation : https://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/aws-properties-cloudfront-distribution-customerrorresponse.html#cfn-cloudfront-distribution-customerrorresponse-errorcachingminttl
       webAclId: props.webACLId,
     };
@@ -855,7 +856,7 @@ export class CloudFrontWebDistribution extends cdk.Construct implements IDistrib
         const validProtocols = this.VALID_SSL_PROTOCOLS[sslSupportMethod as SSLMethod];
 
         if (validProtocols.indexOf(minimumProtocolVersion.toString()) === -1) {
-          // tslint:disable-next-line:max-line-length
+          // eslint-disable-next-line max-len
           throw new Error(`${minimumProtocolVersion} is not compabtible with sslMethod ${sslSupportMethod}.\n\tValid Protocols are: ${validProtocols.join(', ')}`);
         }
       }

packages/@aws-cdk/aws-cloudfront/test/integ.cloudfront-geo-restrictions.ts

@@ -16,7 +16,7 @@ new cloudfront.CloudFrontWebDistribution(stack, 'MyDistribution', {
       s3OriginSource: {
         s3BucketSource: sourceBucket,
       },
-      behaviors : [ {isDefaultBehavior: true}],
+      behaviors: [ {isDefaultBehavior: true}],
     },
   ],
   geoRestriction: cloudfront.GeoRestriction.whitelist('US', 'UK'),

packages/@aws-cdk/aws-cloudfront/test/integ.cloudfront-ipv6-disabled.ts

@@ -17,7 +17,7 @@ new cloudfront.CloudFrontWebDistribution(stack, 'MyDistribution', {
       s3OriginSource: {
         s3BucketSource: sourceBucket,
       },
-      behaviors : [ {isDefaultBehavior: true}],
+      behaviors: [ {isDefaultBehavior: true}],
     },
   ],
   enableIpV6: false,

packages/@aws-cdk/aws-cloudfront/test/integ.cloudfront-lambda-association.ts

@@ -27,7 +27,7 @@ new cloudfront.CloudFrontWebDistribution(stack, 'MyDistribution', {
       s3OriginSource: {
         s3BucketSource: sourceBucket,
       },
-      behaviors : [ {isDefaultBehavior: true, lambdaFunctionAssociations: [{
+      behaviors: [ {isDefaultBehavior: true, lambdaFunctionAssociations: [{
         eventType: cloudfront.LambdaEdgeEventType.ORIGIN_REQUEST,
         lambdaFunction: lambdaVersion,
       }]}],

packages/@aws-cdk/aws-cloudfront/test/integ.cloudfront.ts

@@ -17,7 +17,7 @@ new cloudfront.CloudFrontWebDistribution(stack, 'MyDistribution', {
       s3OriginSource: {
         s3BucketSource: sourceBucket,
       },
-      behaviors : [ {isDefaultBehavior: true}],
+      behaviors: [ {isDefaultBehavior: true}],
     },
   ],
 });

packages/@aws-cdk/aws-cloudfront/test/test.basic.ts

@@ -15,7 +15,7 @@ import {
   ViewerProtocolPolicy,
 } from '../lib';
 
-// tslint:disable:object-literal-key-quotes
+/* eslint-disable quote-props */
 
 export = {
 
@@ -809,7 +809,7 @@ export = {
       originConfigs: [
         {
           s3OriginSource: { s3BucketSource: sourceBucket },
-          behaviors : [
+          behaviors: [
             {
               isDefaultBehavior: true, lambdaFunctionAssociations: [
                 {
@@ -859,7 +859,7 @@ export = {
       originConfigs: [
         {
           s3OriginSource: { s3BucketSource: sourceBucket },
-          behaviors : [
+          behaviors: [
             {
               isDefaultBehavior: true, lambdaFunctionAssociations: [
                 {
@@ -878,7 +878,7 @@ export = {
   },
 
   'geo restriction': {
-    'success' : {
+    'success': {
       'whitelist'(test: Test) {
         const stack = new cdk.Stack();
         const sourceBucket = new s3.Bucket(stack, 'Bucket');