Skip to content

Commit

Permalink
Skip jsdoc checks on windows due to nodejs/node#52554
Browse files Browse the repository at this point in the history
Failure is like:
  1) JSDoc contains all methods
       "before all" hook for "Database":
     Error: spawn EINVAL
      at ChildProcess.spawn (node:internal/child_process:414:11)
      at spawn (node:child_process:761:9)
      at execFile (node:child_process:351:17)
      at jsdocParser (node_modules\jsdoc3-parser\index.js:58:3)
      at node:internal/util:441:7
      at new Promise (<anonymous>)
      at jsdocParser (node:internal/util:427:12)

unsure whether jsdocParser can be bumped, but looks to be broken still.
  • Loading branch information
carlopi committed Apr 19, 2024
1 parent 0cf7eb8 commit 2d4ccb6
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions test/jsdoc.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -24,6 +24,7 @@ export interface Node {
scope: string;
}
describe("JSDoc contains all methods", () => {
if (process.platform == 'win32') this.skip();

Check failure on line 27 in test/jsdoc.test.ts

View workflow job for this annotation

GitHub Actions / node.js Windows (21, false)

Object is possibly 'undefined'.
let docs: Node[];
before(async () => {
docs = await promisify(jsdoc)(require.resolve("../lib/duckdb"));
Expand Down

0 comments on commit 2d4ccb6

Please sign in to comment.