From package scripts to makefiles

Author: vovacodes

.github/workflows/test.yml

@@ -25,5 +25,6 @@ jobs:
                   yarn install
                   make build
                   make test
+                  make lint
               env:
                   CI: true

Makefile

@@ -13,8 +13,13 @@ dev: build
 	"make -C packages/react-sunbeam ${@}" \
 	"make -C packages/demo ${@}"
 
+lint:
+	make -C packages/react-sunbeam $@
+	make -C packages/demo $@
+
 type-check:
-	yarn tsc --noEmit
+	make -C packages/react-sunbeam $@
+	make -C packages/demo $@
 
 test:
 	make -C packages/react-sunbeam $@

package.json

@@ -3,63 +3,19 @@
     "workspaces": [
         "packages/*"
     ],
-    "scripts": {
-        "build": "lerna run build",
-        "commit": "git-cz",
-        "commit:retry": "git-cz --retry",
-        "dev": "yarn build && lerna run --parallel --stream dev",
-        "clean": "lerna run clean",
-        "lint": "eslint packages/react-sunbeam/src --ext .ts,.tsx && eslint packages/demo/src --ext .ts,.tsx",
-        "release": "yarn build && lerna publish --conventional-commits",
-        "test": "lerna run test",
-        "type-check": "lerna run type-check",
-        "uninstall": "lerna clean && rimraf ./node_modules"
-    },
-    "husky": {
-        "hooks": {
-            "pre-commit": "pretty-quick --staged && yarn type-check && yarn lint && yarn test",
-            "commit-msg": "commitlint -E HUSKY_GIT_PARAMS",
-            "pre-push": "yarn type-check && yarn lint && yarn test"
-        }
-    },
-    "commitlint": {
-        "extends": [
-            "@commitlint/config-conventional"
-        ],
-        "rules": {
-            "scope-case": [
-                0
-            ],
-            "header-max-length": [
-                2,
-                "always",
-                100
-            ]
-        }
-    },
-    "config": {
-        "commitizen": {
-            "path": "cz-conventional-changelog"
-        }
-    },
     "repository": "git+https://github.com/vovaguguiev/react-sunbeam.git",
     "author": "Vova Guguiev <vladimir.guguiev@gmail.com>",
     "devDependencies": {
-        "@commitlint/cli": "^11.0.0",
-        "@commitlint/config-conventional": "^11.0.0",
         "@typescript-eslint/eslint-plugin": "^4.7.0",
         "@typescript-eslint/parser": "^4.7.0",
-        "commitizen": "^4.2.2",
-        "cz-conventional-changelog": "^3.3.0",
         "eslint": "^7.13.0",
         "eslint-config-prettier": "^6.15.0",
         "eslint-plugin-import": "^2.22.1",
         "eslint-plugin-react": "^7.21.5",
         "eslint-plugin-react-hooks": "^4.2.0",
-        "husky": "^4.3.0",
-        "lerna": "^3.22.1",
-        "prettier": "^2.1.2",
-        "pretty-quick": "^3.1.0",
-        "rimraf": "^3.0.2"
+        "jest": "^26.6.3",
+        "jest-ts-webcompat-resolver": "^1.0.0",
+        "ts-jest": "^26.4.4",
+        "typescript": "^4.1.2"
     }
 }

packages/demo/Makefile

@@ -11,7 +11,10 @@ dev:
 	 	'tsc --watch --preserveWatchOutput' \
 		'deno run --allow-net --allow-read https://deno.land/x/denoliver@2.1.0/mod.ts .. -p 3333 --entry /demo/index.html'
 
+lint:
+	yarn eslint src --ext .ts,.tsx
+
 type-check:
 	yarn tsc --noEmit
 
-PHONY: build dev type-check
+PHONY: build dev lint type-check

packages/demo/package.json

@@ -8,7 +8,6 @@
         "concurrently": "^6.0.0",
         "framer-motion": "^3.10.3",
         "react-router": "^5.2.0",
-        "@types/react-router-dom": "5.1.6",
-        "typescript": "^4.1.2"
+        "@types/react-router-dom": "5.1.6"
     }
 }

packages/demo/src/apps/SettingsMenu/Picker.tsx

@@ -49,13 +49,16 @@ export function Picker({
             }}
         >
             {({ focused, path }) => {
+                // This callback is called during rendering and it never changes so we can safely call hooks inside of it.
+                /* eslint-disable react-hooks/rules-of-hooks */
                 const focusManager = useFocusManager()
                 const prevOpen = usePrevious(open, open)
                 useEffect(() => {
                     if (!open || prevOpen === open) return
 
                     focusManager.setFocus(path.concat(focusKeyByValue[selectedOption.props.value]))
                 })
+                /* eslint-enable react-hooks/rules-of-hooks */
 
                 return (
                     <div

packages/react-sunbeam/Makefile

@@ -12,7 +12,10 @@ dev:
 type-check:
 	yarn tsc --noEmit
 
+lint:
+	yarn eslint src --ext .ts,.tsx
+
 test:
 	yarn jest
 
-PHONY: clean build dev type-check test
+PHONY: clean build dev lint type-check test

packages/react-sunbeam/package.json

@@ -24,12 +24,8 @@
         "@types/jest": "^26.0.15",
         "@types/react": "^17.0.0",
         "@types/react-dom": "^17.0.0",
-        "jest": "^26.6.3",
-        "jest-ts-webcompat-resolver": "^1.0.0",
         "react": "^17.0.1",
-        "react-dom": "^17.0.1",
-        "ts-jest": "^26.4.4",
-        "typescript": "^4.1.2"
+        "react-dom": "^17.0.1"
     },
     "peerDependencies": {
         "react": "^16 || ^17",