chore: add precommit and prepush

Author: robertherber

.eslintrc

@@ -3,6 +3,8 @@
   "ignorePatterns": ["lib", "/utils", "/hooks", "/contexts", "/components", "/primitives", "/index.js", "index.d.ts", "/stories", "/example-app", "/storybook-static"],
   "rules": {
     "import/no-extraneous-dependencies": 0,
-    "react/boolean-prop-naming": 0
+    "react/boolean-prop-naming": 0,
+    "unicorn/expiring-todo-comments": 0,
+    "@typescript-eslint/sort-type-union-intersection-members": 0
   }
 }

.github/workflows/publish-auto.yml

@@ -23,4 +23,4 @@ jobs:
       npm_token: ${{ secrets.npm_token }}
     with:
       description: Autorelease
-      release_type: patch
+      release_type: patch

.github/workflows/test.yml

@@ -27,7 +27,7 @@ jobs:
       # Checks-out your repository under $GITHUB_WORKSPACE, so your job can access it
       - uses: actions/checkout@v3
         id: checkout
-        
+
       - uses: pnpm/action-setup@v2
 
       - name: Setup Node.js environment
@@ -55,7 +55,7 @@ jobs:
     steps:
       # Checks-out your repository under $GITHUB_WORKSPACE, so your job can access it
       - uses: actions/checkout@v3
-        
+
       - uses: pnpm/action-setup@v2
 
       - name: Setup Node.js environment
@@ -85,7 +85,7 @@ jobs:
       # Checks-out your repository under $GITHUB_WORKSPACE, so your job can access it
       - uses: actions/checkout@v3
         id: checkout
-        
+
       - uses: pnpm/action-setup@v2
 
       - name: Setup Node.js environment
@@ -115,7 +115,7 @@ jobs:
       # Checks-out your repository under $GITHUB_WORKSPACE, so your job can access it
       - uses: actions/checkout@v3
         id: checkout
-        
+
       - uses: pnpm/action-setup@v2
 
       - name: Setup Node.js environment
@@ -130,4 +130,4 @@ jobs:
         run: pnpm install --frozen-lockfile
 
       - name: Lint
-        run: pnpm lint-only
+        run: pnpm lint-only

.husky/pre-commit

@@ -0,0 +1,4 @@
+#!/usr/bin/env sh
+. "$(dirname -- "$0")/_/husky.sh"
+
+pnpm lint-staged

.husky/pre-push

@@ -0,0 +1,4 @@
+#!/usr/bin/env sh
+. "$(dirname -- "$0")/_/husky.sh"
+
+pnpm test

package.json

@@ -28,6 +28,13 @@
     "build-storybook": "build-storybook",
     "release": "yarn test && npx release-it"
   },
+  "lint-staged": {
+    "*.ts": "eslint --fix",
+    "*.js": "eslint --fix",
+    "*.json": "eslint --fix",
+    "*.graphql": "eslint --fix",
+    "*.yml": "eslint --fix"
+  },
   "publishConfig": {
     "access": "public",
     "registry": "https://registry.npmjs.org/"
@@ -92,6 +99,7 @@
     "jest": "27",
     "jest-expo": "^45.0.1",
     "jwt-decode": "^3.1.2",
+    "lint-staged": "^13.2.3",
     "lodash": "^4.17.21",
     "react": "17",
     "react-dom": "17",

pnpm-lock.yaml

@@ -1,3 +1,4 @@
+/* eslint-disable react/destructuring-assignment */
 import { MaterialCommunityIcons } from '@expo/vector-icons'
 import React, {
   useContext,

src/components/GenericError.tsx

@@ -21,7 +21,8 @@ export const NativePortal: React.FC<Props> = ({
   return (
     <Portal>
       <FullWindowOverlay
-        pointerEvents={pointerEvents}
+        // @ts-expect-error not sure why this is happening
+        pointerEvents={pointerEvents as unknown}
         style={style}
       >
         { children }

src/components/NativePortal.ios.tsx

@@ -5,8 +5,8 @@ import {
 
 import type { RefreshControlProps } from 'react-native'
 
-type Props = Omit<RefreshControlProps, 'onRefresh' | 'refreshing'> & {
-  readonly onRefetch: (() => Promise<unknown> | unknown) | undefined,
+type Props<T = unknown> = Omit<RefreshControlProps, 'onRefresh' | 'refreshing'> & {
+  readonly onRefetch: (() => Promise<T> | T) | undefined,
   readonly onStatusChange?: (isLoading: boolean) => void,
 }