fix: issues found by code scanning

zwave-js · Jul 9, 2024 · deee0a0 · deee0a0
1 parent 7d65f38
commit deee0a0
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 7 deletions.
diff --git a/.github/action-scripts/generateTypedDocs.js b/.github/action-scripts/generateTypedDocs.js
@@ -19,7 +19,7 @@ const checkPaths = ["docs/", "packages/*/*.api.md"];
  * @param {{github: Github, context: Context}} param
  */
 async function main(param) {
-	const { github, context } = param;
+	const { github } = param;
 
 	// check if our local working copy has any changes in the docs directory
 	const isChanged = !!(await exec.exec(

diff --git a/.github/bot-scripts/ensureLogfileInDiscussion.js b/.github/bot-scripts/ensureLogfileInDiscussion.js
@@ -88,11 +88,6 @@ Please consider uploading a logfile that captures your problem. As a reminder, h
 		}
 	}
 
-	const options = {
-		owner: context.repo.owner,
-		repo: context.repo.repo,
-	};
-
 	// TODO: Consider if we want to delete outdated comments. That may delete replies as well though.
 	if (hasZjsLog || !message) return;
 

diff --git a/packages/eslint-plugin/src/utils/titleAndSentenceCase.ts b/packages/eslint-plugin/src/utils/titleAndSentenceCase.ts
@@ -177,7 +177,7 @@ function sentenceCaseWord(word: string, isFirstWord: boolean): string {
 		const exception = titleCaseExceptions.find(
 			(ex) => ex.toLowerCase() === lowercase,
 		);
-		if (exception) exception;
+		if (exception) return exception;
 	}
 	if (
 		titleCaseIgnored.some((re) => re.test(word))