Add linting for Typescript (#2)

* feat(package): extend eslint:recommended

* dep: add prettier to project

* feat(package): add support for linting typescript

* refactor: rename file names

Author: antoine-cottineau

.prettierrc

@@ -0,0 +1 @@
+{}

.vscode/settings.json

@@ -0,0 +1,4 @@
+{
+  "editor.defaultFormatter": "esbenp.prettier-vscode",
+  "editor.formatOnSave": true
+}

README.md

@@ -1,4 +1,5 @@
 # react-native-project-config
+
 Monorepo with packages for setting up ESLint, Typescript, Prettier and Jest.
 
 ## Presentation
@@ -12,4 +13,4 @@ There is also an example app under `packages/example-app` whose goal is to demon
 ## Commands
 
 - `yarn lerna run start`: run the 'start' script in all packages (currently only present in `example-app`),
-- `yarn lerna run lint`: run the 'lint' script in all packages (currently only present in `example-app`).
+- `yarn lerna run lint`: run the 'lint' script in all packages (currently only present in `example-app`).

package.json

@@ -5,6 +5,7 @@
     "packages/*"
   ],
   "devDependencies": {
-    "lerna": "^6.3.0"
+    "lerna": "^6.3.0",
+    "prettier": "^2.8.1"
   }
 }

packages/eslint/index.js

@@ -1,7 +1,13 @@
 "use strict";
 
 module.exports = {
-  rules: {
-    "no-console": "error",
+  env: {
+    node: true,
   },
+  extends: ["eslint:recommended", "plugin:@typescript-eslint/recommended"],
+  parser: "@typescript-eslint/parser",
+  parserOptions: {
+    ecmaVersion: "latest",
+  },
+  plugins: ["@typescript-eslint"],
 };

packages/eslint/package.json

@@ -2,5 +2,12 @@
   "name": "@bam.tech/eslint-config",
   "version": "0.0.0",
   "main": "index.js",
-  "license": "MIT"
+  "license": "MIT",
+  "peerDependencies": {
+    "@typescript-eslint/eslint-plugin": ">= 5",
+    "eslint": ">= 8"
+  },
+  "devDependencies": {
+    "@typescript-eslint/parser": "^5.48.0"
+  }
 }

packages/example-app/.eslintrc

@@ -1,9 +1,3 @@
 {
-  "env": {
-    "es6": true
-  },
-  "extends": "@bam.tech/eslint-config",
-  "parserOptions": {
-    "ecmaVersion": "latest"
-  }
+  "extends": "@bam.tech/eslint-config"
 }

packages/example-app/break-eslint-recommended-rules.js

@@ -0,0 +1,11 @@
+"use strict";
+
+console.log(
+  "This is a console.log statement to see it the linter produces an error"
+);
+
+// This should trigger two errors breaking eslint:recommended rules:
+// no-constant-condition
+// no-empty
+if (true) {
+}

packages/example-app/break-typescript-eslint-rules.ts

@@ -0,0 +1,4 @@
+// This should trigger two errors breaking typescript-eslint rules:
+// @typescript-eslint/no-unused-vars
+// @typescript-eslint/ban-types
+const str: String = "foo";