fix(eslint-config): replace gts with prettier and eslint (#1439)

* fix(eslint-config): replace gts with prettier and eslint recommended config

* fix(eslint-config): using the core repo's configuration

---------

Co-authored-by: Haddas Bronfman <85441461+haddasbronfman@users.noreply.github.com>

Author: samimusallam

archive/opentelemetry-browser-extension-autoinjection/package.json

@@ -44,7 +44,6 @@
     "eslint-plugin-header": "3.1.1",
     "eslint-plugin-import": "2.25.4",
     "eslint-plugin-json5": "0.1.4",
-    "gts": "3.1.0",
     "html-webpack-plugin": "5.3.2",
     "jimp": "0.16.1",
     "jsdom": "15.2.1",

detectors/node/opentelemetry-resource-detector-alibaba-cloud/package.json

@@ -47,7 +47,6 @@
     "@types/mocha": "8.2.3",
     "@types/node": "18.11.7",
     "@types/sinon": "10.0.2",
-    "gts": "3.1.0",
     "mocha": "7.2.0",
     "nock": "12.0.3",
     "nyc": "15.1.0",

detectors/node/opentelemetry-resource-detector-aws/package.json

@@ -46,7 +46,6 @@
     "@types/mocha": "8.2.3",
     "@types/node": "18.11.7",
     "@types/sinon": "10.0.2",
-    "gts": "3.1.0",
     "mocha": "7.2.0",
     "nock": "12.0.3",
     "nyc": "15.1.0",

detectors/node/opentelemetry-resource-detector-container/package.json

@@ -41,7 +41,6 @@
     "@types/node": "^18.0.0",
     "@types/sinon": "10.0.2",
     "eslint-plugin-header": "^3.1.1",
-    "gts": "^3.1.0",
     "mocha": "7.2.0",
     "nock": "12.0.3",
     "nyc": "15.1.0",

detectors/node/opentelemetry-resource-detector-gcp/package.json

@@ -45,7 +45,7 @@
     "@opentelemetry/contrib-test-utils": "^0.33.2",
     "@types/mocha": "8.2.3",
     "@types/node": "18.11.7",
-    "gts": "3.1.0",
+    "@types/semver": "7.3.8",
     "mocha": "7.2.0",
     "nock": "12.0.3",
     "nyc": "15.1.0",

detectors/node/opentelemetry-resource-detector-github/package.json

@@ -49,7 +49,6 @@
     "@types/mocha": "8.2.3",
     "@types/node": "18.11.7",
     "@types/sinon": "10.0.2",
-    "gts": "3.1.0",
     "mocha": "7.2.0",
     "nyc": "15.1.0",
     "rimraf": "4.2.0",

detectors/node/opentelemetry-resource-detector-instana/package.json

@@ -47,7 +47,6 @@
     "@types/semver": "7.3.8",
     "mocha": "7.2.0",
     "nock": "12.0.3",
-    "gts": "3.1.0",
     "nyc": "15.1.0",
     "rimraf": "4.2.0",
     "ts-mocha": "10.0.0",

eslint.config.js

@@ -2,50 +2,64 @@ module.exports = {
   plugins: [
     "@typescript-eslint",
     "header",
-    "import"
-  ],
-  extends: [
-      "./node_modules/gts",
+    "node",
+    "prettier"
   ],
+  extends: ["eslint:recommended", "plugin:@typescript-eslint/recommended", "plugin:prettier/recommended"],
   parser: "@typescript-eslint/parser",
   parserOptions: {
-      "project": "./tsconfig.json"
+    "project": "./tsconfig.json"
   },
   rules: {
-    "quotes": [2, "single", { "avoidEscape": true }],
-    "@typescript-eslint/no-this-alias": "off",
+    "quotes": ["error", "single", { "avoidEscape": true }],
     "eqeqeq": "off",
     "prefer-rest-params": "off",
-    "@typescript-eslint/naming-convention": [
-        "error",
-        {
-          "selector": "memberLike",
-          "modifiers": ["private", "protected"],
-          "format": ["camelCase"],
-          "leadingUnderscore": "require"
-        }
-    ],
-    "@typescript-eslint/no-inferrable-types": ["error", { ignoreProperties: true }],
-    "arrow-parens": ["error", "as-needed"],
-    "prettier/prettier": ["error", { "singleQuote": true, "arrowParens": "avoid" }],
+    "no-shadow": "off",
     "node/no-deprecated-api": ["warn"],
-    "header/header": [2, "block", [{
-        pattern: / \* Copyright The OpenTelemetry Authors[\r\n]+ \*[\r\n]+ \* Licensed under the Apache License, Version 2\.0 \(the \"License\"\);[\r\n]+ \* you may not use this file except in compliance with the License\.[\r\n]+ \* You may obtain a copy of the License at[\r\n]+ \*[\r\n]+ \*      https:\/\/www\.apache\.org\/licenses\/LICENSE-2\.0[\r\n]+ \*[\r\n]+ \* Unless required by applicable law or agreed to in writing, software[\r\n]+ \* distributed under the License is distributed on an \"AS IS\" BASIS,[\r\n]+ \* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied\.[\r\n]+ \* See the License for the specific language governing permissions and[\r\n]+ \* limitations under the License\./gm,
-        template:
+    "header/header": ["error", "block", [{
+      pattern: / \* Copyright The OpenTelemetry Authors[\r\n]+ \*[\r\n]+ \* Licensed under the Apache License, Version 2\.0 \(the \"License\"\);[\r\n]+ \* you may not use this file except in compliance with the License\.[\r\n]+ \* You may obtain a copy of the License at[\r\n]+ \*[\r\n]+ \*      https:\/\/www\.apache\.org\/licenses\/LICENSE-2\.0[\r\n]+ \*[\r\n]+ \* Unless required by applicable law or agreed to in writing, software[\r\n]+ \* distributed under the License is distributed on an \"AS IS\" BASIS,[\r\n]+ \* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied\.[\r\n]+ \* See the License for the specific language governing permissions and[\r\n]+ \* limitations under the License\./gm,
+      template:
         `\n * Copyright The OpenTelemetry Authors\n *\n * Licensed under the Apache License, Version 2.0 (the "License");\n * you may not use this file except in compliance with the License.\n * You may obtain a copy of the License at\n *\n *      https://www.apache.org/licenses/LICENSE-2.0\n *\n * Unless required by applicable law or agreed to in writing, software\n * distributed under the License is distributed on an "AS IS" BASIS,\n * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.\n * See the License for the specific language governing permissions and\n * limitations under the License.\n `
-    }]],
-    "import/no-extraneous-dependencies": ["error", { devDependencies: ["test/**/*.ts"] }],
+    }]]
   },
   overrides: [
     {
-      "files": ["test/**/*.ts"],
-      "rules": {
+      files: ['*.ts'],
+      rules: {
+        "@typescript-eslint/no-this-alias": "off",
+        "@typescript-eslint/naming-convention": [
+          "error",
+          {
+            "selector": "memberLike",
+            "modifiers": ["private", "protected"],
+            "format": ["camelCase"],
+            "leadingUnderscore": "require"
+          }
+        ],
+        "@typescript-eslint/no-var-requires": "off",
+        "@typescript-eslint/no-inferrable-types": ["error", { ignoreProperties: true }],
+        "@typescript-eslint/no-empty-function": ["off"],
+        "@typescript-eslint/ban-types": ["warn", {
+          "types": {
+            "Function": null,
+          }
+        }],
+        "@typescript-eslint/no-shadow": ["warn"],
+      }
+    },
+    {
+      files: ["test/**/*.ts"],
+      rules: {
         "no-empty": "off",
         "@typescript-eslint/ban-ts-ignore": "off",
         "@typescript-eslint/no-empty-function": "off",
         "@typescript-eslint/no-explicit-any": "off",
         "@typescript-eslint/no-unused-vars": "off",
-        "@typescript-eslint/no-var-requires": "off"
+        "@typescript-eslint/no-var-requires": "off",
+        "@typescript-eslint/no-shadow": ["off"],
+        "@typescript-eslint/no-floating-promises": ["off"],
+        "@typescript-eslint/no-non-null-assertion": ["off"],
+        "@typescript-eslint/explicit-module-boundary-types": ["off"]
       }
     }
   ]

metapackages/auto-instrumentations-node/package.json

@@ -41,7 +41,6 @@
     "@types/mocha": "7.0.2",
     "@types/node": "18.11.7",
     "@types/sinon": "10.0.2",
-    "gts": "3.1.0",
     "mocha": "7.2.0",
     "nyc": "15.1.0",
     "rimraf": "4.2.0",

metapackages/auto-instrumentations-web/package.json

@@ -40,7 +40,6 @@
     "@types/sinon": "10.0.2",
     "@types/webpack-env": "1.16.2",
     "babel-loader": "8.2.2",
-    "gts": "3.1.0",
     "@jsdevtools/coverage-istanbul-loader": "3.0.5",
     "karma": "6.3.16",
     "karma-chrome-launcher": "^3.1.1",