added instructions to developer file

valkey-io · Feb 20, 2024 · 3a9d61b · 3a9d61b
1 parent 6eeafbe
commit 3a9d61b
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 3 deletions.
diff --git a/.github/workflows/lint-ts.yml b/.github/workflows/lint-ts.yml
@@ -40,7 +40,17 @@ jobs:
 
             - name: lint ts
               run: |
-                  npm install @typescript-eslint/parser @typescript-eslint/eslint-plugin eslint-plugin-tsdoc eslint typescript eslint-plugin-import@latest eslint-config-prettier prettier
+                  npm install @typescript-eslint/parser @typescript-eslint/eslint-plugin eslint-plugin-tsdoc eslint typescript eslint-plugin-import@latest eslint-config-prettier
                   npm i
                   npx eslint .
-                  npm run prettier:check:ci
+
+            - name: Iterate over folders and prettier
+                run: |
+                folders=("node" "benchmarks/node" "benchmarks/utilities")
+                for folder in "${folders[@]}"; do
+                    echo "Linting ${folder}..."
+                    cd "${folder}"
+                    npm install
+                    npm run prettier:check:ci
+                    cd ..
+                done
diff --git a/node/DEVELOPER.md b/node/DEVELOPER.md
@@ -149,9 +149,11 @@ Development on the Node wrapper may involve changes in either the TypeScript or
 1. TypeScript
     ```bash
     # Run from the `node` folder
-    npm install eslint-plugin-import@latest  @typescript-eslint/parser @typescript-eslint/eslint-plugin eslint-plugin-tsdoc eslint typescript eslint-plugin-import@latest eslint-config-prettier
+    npm install eslint-plugin-import@latest  @typescript-eslint/parser @typescript-eslint/eslint-plugin eslint-plugin-tsdoc eslint typescript eslint-plugin-import@latest eslint-config-prettier prettier
     npm i
     npx eslint . --max-warnings=0
+    npm run prettier:format
+    npm run prettier:check:ci
     ```
 2. Rust
     ```bash