🏗 Enable gulp prettify on several non-JS files (and auto-fix them) (a…

…mpproject#25085)
prateekbh · Oct 16, 2019 · e231dea · e231dea
1 parent f515760
commit e231dea
Show file tree

Hide file tree

Showing 219 changed files with 23,606 additions and 1,073 deletions.
diff --git a/.eslintrc b/.eslintrc
@@ -15,7 +15,7 @@
     "es6": true,
     "browser": true
   },
- "parserOptions": {
+  "parserOptions": {
     "ecmaVersion": 6,
     "sourceType": "module"
   },
@@ -33,7 +33,7 @@
         "file": "fileoverview",
         "returns": "return"
       },
-      "allowOverrideWithoutParam" : true
+      "allowOverrideWithoutParam": true
     }
   },
   "rules": {
@@ -43,25 +43,28 @@
     "curly": 2,
     "google-camelcase/google-camelcase": 2,
     "jsdoc/check-param-names": 2,
-    "jsdoc/check-tag-names": [2, {
-      "definedTags": [
-        "closurePrimitive",
-        "deprecated",
-        "dict",
-        "export",
-        "final",
-        "nocollapse",
-        "noinline",
-        "package",
-        "record",
-        "restricted",
-        "struct",
-        "suppress",
-        "template",
-        "visibleForTesting"
-      ]
-    }],
-    "jsdoc/check-types": [2, { "noDefaults": true }],
+    "jsdoc/check-tag-names": [
+      2,
+      {
+        "definedTags": [
+          "closurePrimitive",
+          "deprecated",
+          "dict",
+          "export",
+          "final",
+          "nocollapse",
+          "noinline",
+          "package",
+          "record",
+          "restricted",
+          "struct",
+          "suppress",
+          "template",
+          "visibleForTesting"
+        ]
+      }
+    ],
+    "jsdoc/check-types": [2, {"noDefaults": true}],
     "jsdoc/require-param": 2,
     "jsdoc/require-param-name": 2,
     "jsdoc/require-param-type": 2,
@@ -111,7 +114,7 @@
     "no-eval": 2,
     "no-extend-native": 2,
     "no-extra-bind": 2,
-    "no-implicit-coercion": [2, { "boolean": false }],
+    "no-implicit-coercion": [2, {"boolean": false}],
     "no-implied-eval": 2,
     "no-iterator": 2,
     "no-lone-blocks": 2,
@@ -123,16 +126,23 @@
     "no-sequences": 2,
     "no-throw-literal": 2,
     "no-unused-expressions": 0,
-    "no-unused-vars": [2, {
-      "argsIgnorePattern": "^(var_args$|opt_|unused)",
-      "varsIgnorePattern": "(AmpElement|Def|Interface)$"
-    }],
+    "no-unused-vars": [
+      2,
+      {
+        "argsIgnorePattern": "^(var_args$|opt_|unused)",
+        "varsIgnorePattern": "(AmpElement|Def|Interface)$"
+      }
+    ],
     "no-useless-call": 2,
     "no-useless-concat": 2,
     "no-undef": 2,
     "no-var": 2,
-    "no-warning-comments": [2, { "terms": ["do not submit"], "location": "anywhere" }],
-    "notice/notice": [2,
+    "no-warning-comments": [
+      2,
+      {"terms": ["do not submit"], "location": "anywhere"}
+    ],
+    "notice/notice": [
+      2,
       {
         "mustMatch": "Copyright 20\\d{2} The AMP HTML Authors\\.",
         "templateFile": "build-system/common/LICENSE-TEMPLATE.txt",
@@ -141,24 +151,30 @@
         }
       }
     ],
-    "object-shorthand": [2, "properties", { "avoidQuotes": true }],
+    "object-shorthand": [2, "properties", {"avoidQuotes": true}],
     "prefer-const": 2,
     "prettier/prettier": 2,
     "radix": 2,
-    "require-jsdoc": [2, {
-      "require": {
-        "FunctionDeclaration": true,
-        "MethodDefinition": true,
-        "ClassDeclaration": false,
-        "ArrowFunctionExpression": false,
-        "FunctionExpression": false
+    "require-jsdoc": [
+      2,
+      {
+        "require": {
+          "FunctionDeclaration": true,
+          "MethodDefinition": true,
+          "ClassDeclaration": false,
+          "ArrowFunctionExpression": false,
+          "FunctionExpression": false
+        }
+      }
+    ],
+    "sort-imports-es6-autofix/sort-imports-es6": [
+      2,
+      {
+        "ignoreCase": false,
+        "ignoreMemberSort": false,
+        "memberSyntaxSortOrder": ["none", "all", "multiple", "single"]
       }
-    }],
-    "sort-imports-es6-autofix/sort-imports-es6": [2, {
-      "ignoreCase": false,
-      "ignoreMemberSort": false,
-      "memberSyntaxSortOrder": ["none", "all", "multiple", "single"]
-    }],
+    ],
     "sort-requires/sort-requires": 2
   },
   "overrides": [

diff --git a/.lando.yml b/.lando.yml
@@ -43,35 +43,35 @@ tooling:
     service: appserver
   gulp:
     service: appserver
-    cmd: "yarn gulp"
+    cmd: 'yarn gulp'
   watch:
     service: appserver
-    cmd: "yarn gulp watch"
-    description: "Watches for changes in files, re-builds when detected"
+    cmd: 'yarn gulp watch'
+    description: 'Watches for changes in files, re-builds when detected'
   lint:
     service: appserver
-    cmd: "yarn gulp lint"
-    description: "Validates against Google Closure Linter"
+    cmd: 'yarn gulp lint'
+    description: 'Validates against Google Closure Linter'
   check-types:
     service: appserver
-    cmd: "yarn gulp check-types"
-    description: "Check source code for JS type errors"
+    cmd: 'yarn gulp check-types'
+    description: 'Check source code for JS type errors'
   dist:
     service: appserver
-    cmd: "yarn gulp dist"
-    description: "Build production binaries"
+    cmd: 'yarn gulp dist'
+    description: 'Build production binaries'
   unit:
     service: appserver
-    cmd: "yarn gulp unit"
-    description: "Run unit tests"
+    cmd: 'yarn gulp unit'
+    description: 'Run unit tests'
   integration:
     service: appserver
-    cmd: "yarn gulp integration"
-    description: "Run integration tests"
+    cmd: 'yarn gulp integration'
+    description: 'Run integration tests'
   e2e:
     service: appserver
-    cmd: "yarn gulp e2e"
-    description: "Runs e2e tests"
+    cmd: 'yarn gulp e2e'
+    description: 'Runs e2e tests'
 
 proxy:
   appserver:

diff --git a/.lgtm.yml b/.lgtm.yml
@@ -1,15 +1,15 @@
 path_classifiers:
   docs:
-    - "**/*.md"
+    - '**/*.md'
   test:
-    - "**/test/**/*.js"
-    - "**/testing/**/*.js"
+    - '**/test/**/*.js'
+    - '**/testing/**/*.js'
   third_party:
-    - "third_party/**/*.*"
+    - 'third_party/**/*.*'
   externs:
-    - "**/*.extern.js"
+    - '**/*.extern.js'
   build-system:
-    - "build-system/**/*.*"
+    - 'build-system/**/*.*'
 extraction:
   javascript:
     index:

diff --git a/.prettierrc b/.prettierrc
@@ -6,17 +6,15 @@
   "overrides": [
     {
       "files": "OWNERS",
-      "options": {
-        "parser": "json5",
-        "bracketSpacing": true,
-        "trailingComma": "none"
-      }
+      "options": {"parser": "json5"}
     },
     {
-      "files": ".prettierrc",
-      "options": {
-        "parser": "json"
-      }
+      "files": [".eslintrc", ".prettierrc", ".renovaterc.json", "*.json"],
+      "options": {"parser": "json"}
+    },
+    {
+      "files": [".codecov.yml", ".lando.yml", ".lgtm.yml", ".travis.yml"],
+      "options": {"parser": "yaml"}
     }
   ]
 }
diff --git a/.renovaterc.json b/.renovaterc.json
@@ -1,11 +1,7 @@
 {
-  "extends": [
-    "config:base"
-  ],
+  "extends": ["config:base"],
   "node": {
-    "supportPolicy": [
-      "lts_active"
-    ]
+    "supportPolicy": ["lts_active"]
   },
   "statusCheckVerify": true,
   "commitMessagePrefix": "📦",

diff --git a/.travis.yml b/.travis.yml
@@ -90,15 +90,15 @@ jobs:
       env:
         - CACHE_NAME=E2EJOB
     - stage: experiment
-      name: "Experiment A Tests"
+      name: 'Experiment A Tests'
       script:
         - unbuffer node build-system/pr-check/experiment-tests.js --experiment=experimentA
     - stage: experiment
-      name: "Experiment B Tests"
+      name: 'Experiment B Tests'
       script:
         - unbuffer node build-system/pr-check/experiment-tests.js --experiment=experimentB
     - stage: experiment
-      name: "Experiment C Tests"
+      name: 'Experiment C Tests'
       script:
         - unbuffer node build-system/pr-check/experiment-tests.js --experiment=experimentC
 before_cache:

diff --git a/3p/OWNERS b/3p/OWNERS
@@ -4,7 +4,7 @@
 {
   rules: [
     {
-      owners: [{ name: 'ampproject/wg-ads' }]
-    }
-  ]
+      owners: [{name: 'ampproject/wg-ads'}],
+    },
+  ],
 }
diff --git a/OWNERS b/OWNERS
@@ -6,30 +6,30 @@
   rules: [
     {
       owners: [
-        { name: 'cramforce', requestReviews: false },
-        { name: 'dvoytenko', requestReviews: false },
-        { name: 'jridgewell', requestReviews: false }
-      ]
+        {name: 'cramforce', requestReviews: false},
+        {name: 'dvoytenko', requestReviews: false},
+        {name: 'jridgewell', requestReviews: false},
+      ],
     },
     {
       pattern: '**/validator-*.{protoascii,html,out}',
-      owners: [{ name: 'ampproject/wg-caching', notify: true }]
+      owners: [{name: 'ampproject/wg-caching', notify: true}],
     },
     {
       pattern: '**/*.md',
-      owners: [{ name: 'mrjoro' }, { name: 'ampproject/wg-outreach' }]
+      owners: [{name: 'mrjoro'}, {name: 'ampproject/wg-outreach'}],
     },
     {
       pattern: '{.*,babel.config.js,gulpfile.js}',
-      owners: [{ name: 'ampproject/wg-infra' }]
+      owners: [{name: 'ampproject/wg-infra'}],
     },
     {
       pattern: '{package.json,yarn.lock}',
       owners: [
-        { name: 'ampproject/wg-infra' },
-        { name: 'ampproject/wg-runtime' },
-        { name: 'ampproject/wg-performance' }
-      ]
+        {name: 'ampproject/wg-infra'},
+        {name: 'ampproject/wg-runtime'},
+        {name: 'ampproject/wg-performance'},
+      ],
     },
     {
       // The below rule will be here only temporarily during the transition
@@ -47,9 +47,9 @@
       // and/or the Travis check is implemented.
       pattern: '**/{OWNERS.yaml,OWNERS}',
       owners: [
-        { name: 'rcebulko', notify: true },
-        { name: 'mrjoro', requestReviews: false }
-      ]
-    }
-  ]
+        {name: 'rcebulko', notify: true},
+        {name: 'mrjoro', requestReviews: false},
+      ],
+    },
+  ],
 }
diff --git a/ads/OWNERS b/ads/OWNERS
@@ -4,7 +4,7 @@
 {
   rules: [
     {
-      owners: [{ name: 'ampproject/wg-ads' }]
-    }
-  ]
+      owners: [{name: 'ampproject/wg-ads'}],
+    },
+  ],
 }
diff --git a/ads/google/OWNERS b/ads/google/OWNERS
@@ -4,7 +4,7 @@
 {
   rules: [
     {
-      owners: [{ name: 'ampproject/a4a' }]
-    }
-  ]
+      owners: [{name: 'ampproject/a4a'}],
+    },
+  ],
 }
diff --git a/ads/google/a4a/.eslintrc b/ads/google/a4a/.eslintrc
@@ -1,5 +1,5 @@
 {
   "rules": {
-    "local/no-style-display": 2,
+    "local/no-style-display": 2
   }
 }
diff --git a/build-system/OWNERS b/build-system/OWNERS
@@ -4,7 +4,7 @@
 {
   rules: [
     {
-      owners: [{ name: 'ampproject/wg-infra' }]
-    }
-  ]
+      owners: [{name: 'ampproject/wg-infra'}],
+    },
+  ],
 }