scope prettier/eslint/typecheck to source folder only - excluding exa…

…mple folders

.eslintignore

@@ -1,8 +1,18 @@
 **/node_modules/*
-**/react-native-live-markdown-parser.js
-lib/**/*
-*.test.*
+
+# any js file inside android and ios folders
+android/**/*.js
+ios/**/*.js
+
+# Ignore all files inside subproject folders 
 example/**/*
-*.config.js
+WebExample/**/*
+parser/**/*
+
+# Output of the build process & scripts
+lib/**/*
+scripts/**/*
+
+babel.config.js
 .eslintrc.js
-.prettierrc.js
+.prettierrc.js

.eslintrc.js

@@ -10,6 +10,7 @@ module.exports = {
     'plugin:@typescript-eslint/stylistic-type-checked',
     'plugin:import/typescript',
     'prettier',
+    'plugin:prettier/recommended',
   ],
   plugins: [
     'react',
@@ -25,6 +26,18 @@ module.exports = {
   },
   root: true,
   rules: {
+    'prettier/prettier': [
+      'error',
+      {
+        tabWidth: 2,
+        singleQuote: true,
+        trailingComma: 'all',
+        bracketSpacing: false,
+        arrowParens: 'always',
+        printWidth: 190,
+        singleAttributePerLine: true,
+      },
+    ],
     'rulesdir/prefer-underscore-method': 'off',
     'react/jsx-props-no-spreading': 'off',
     'react/require-default-props': 'off',
@@ -39,7 +52,7 @@ module.exports = {
         "tsx": "never"
       }
    ],
-    'import/no-unresolved': 'error', // no in app
+    'import/no-unresolved': 'error',
     'import/consistent-type-specifier-style': ['error', 'prefer-top-level'],
     'no-use-before-define': 'off',
     '@typescript-eslint/no-use-before-define': 'off', // TODO consider enabling this (currently it reports styles defined at the bottom of the file)

.github/workflows/lint.yml

@@ -28,14 +28,10 @@ jobs:
 
             - run: yarn install --immutable
 
-            - run: yarn lint
-              env:
-                  CI: true
-
 
             - name: Verify there's no Prettier diff
               run: |
-                yarn prettier --loglevel silent
+                yarn lint --fix --quiet
                 if ! git diff --name-only --exit-code; then
                     # shellcheck disable=SC2016
                     echo 'Error: Prettier diff detected! Please run `npm run prettier` and commit the changes.'

package.json

@@ -32,8 +32,6 @@
     "typecheck": "tsc --noEmit",
     "lint": "eslint . --ext .js,.ts,.tsx",
     "clean": "del-cli android/build example/android/build example/android/app/build example/ios/build lib",
-    "prettier": "prettier --write .",
-    "prettier-watch": "onchange \"**/*.js\" -- prettier --write --ignore-unknown {{changed}}",
     "prepare": "bob build",
     "release": "release-it"
   },

src/MarkdownTextInput.web.tsx

@@ -1,14 +1,14 @@
-/* eslint-disable arrow-body-style */
-/* eslint-disable @typescript-eslint/no-empty-interface */
 import {TextInput} from 'react-native';
 
 import React from 'react';
 import type {TextInputProps} from 'react-native';
 
+// eslint-disable-next-line @typescript-eslint/no-empty-interface
 interface MarkdownTextInputProps extends TextInputProps {
   // nothing here
 }
 
+// eslint-disable-next-line arrow-body-style
 const MarkdownTextInput = React.forwardRef<TextInput, MarkdownTextInputProps>((props, ref) => {
   // TODO: add web implementation here
   return (

tsconfig.json

@@ -27,5 +27,5 @@
     "typeRoots": ["node_modules/@types"]
   },
   "include": ["src/**/*", "types/global.d.ts"],
-  "exclude": ["**/node_modules/**/*", "parser/**/*", "**/lib/**/*", "**/__tests__/**/*", "example/src/**/*", "WebExample/**/*"]
+  "exclude": ["**/node_modules/**/*", "parser/**/*", "**/lib/**/*", "example/src/**/*", "WebExample/**/*"]
 }