Merge pull request #48 from CMU-313/add-jshint

feat: add github actions for JSHint

Author: cnk2024

.github/workflows/jshint,yaml .github/workflows/jshint.yaml.github/workflows/jshint,yaml renamed to .github/workflows/jshint.yaml

@@ -25,6 +25,8 @@ jobs:
 
     steps:
       - uses: actions/checkout@v4
+        with:
+          fetch-depth: 0
 
       - run: cp install/package.json package.json
 
@@ -39,4 +41,14 @@ jobs:
           useLockFile: false
 
       - name: Run JShint
-      - run: ./node_modules/.bin/jshint . 
+        run: |
+          git fetch origin main
+          FILES=$(git diff --name-only origin/main...HEAD | grep '\.js$' || true)
+
+          if [ -z "$FILES" ]; then
+            echo "No JS files changed"
+          else
+            echo "Running JSHint on:"
+            echo "$FILES"
+            ./node_modules/.bin/jshint $FILES || echo "JSHint found potential issues in modified files."
+          fi

src/admin/search.js

@@ -139,4 +139,4 @@ module.exports.filterDirectories = filterDirectories;
 module.exports.simplify = simplify;
 module.exports.sanitize = sanitize;
 
-require('../promisify')(module.exports);
+require('../promisify')(module.exports);

src/api/categories.js

@@ -247,4 +247,4 @@ categoriesAPI.setModerator = async (caller, { cid, member, set }) => {
 
 	const privilegeList = await privileges.categories.getUserPrivilegeList();
 	await categoriesAPI.setPrivilege(caller, { cid, privilege: privilegeList, member, set });
-};
+};