Merge pull request #16 from constructive-io/fix/ci-not-detecting-exit-codes

Zetazzz · web-flow · commit 05ab2747357e · 2025-12-19T22:37:14.000+08:00
fix(ci): promise-returning function makes each test always eval true
diff --git a/scripts/test-all-packages.js b/scripts/test-all-packages.js
@@ -128,7 +128,7 @@ function runPgpmCommand(command, dbname, packageName, packagePath, projectRoot)
   return result.success;
 }
 
-async function testPackage(packagePath, projectRoot, useDocker) {
+function testPackage(packagePath, projectRoot, useDocker) {
   const packageName = path.basename(packagePath);
   const dbname = dbsafename(packagePath);
   

Original file line number	Diff line number	Diff line change
`@@ -128,7 +128,7 @@ function runPgpmCommand(command, dbname, packageName, packagePath, projectRoot)`
`128`	`128`	`return result.success;`
`129`	`129`	`}`
`130`	`130`
`131`		`-async function testPackage(packagePath, projectRoot, useDocker) {`
	`131`	`+function testPackage(packagePath, projectRoot, useDocker) {`
`132`	`132`	`const packageName = path.basename(packagePath);`
`133`	`133`	`const dbname = dbsafename(packagePath);`
`134`	`134`