Skip to content

Commit

Permalink
Remove pnpm jsdoc from pnpm test:ci
Browse files Browse the repository at this point in the history
I forgot that 'jsdoc' isn't actually a devDependency. D'oh! We'll get
most of the benefits from `pnpm typecheck` anyway.

`pnpm jsdoc` was mainly to check that the JSDoc comments remained
compatible. I'll consider adding 'jsdoc' as a devDependency, but I'm on
the fence.
  • Loading branch information
mbland committed Jan 9, 2024
1 parent 01a79f6 commit fdf3d58
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@
"scripts": {
"lint": "eslint --color --max-warnings 0 .",
"test": "vitest",
"test:ci": "pnpm lint && pnpm typecheck && pnpm jsdoc && pnpm test:ci:jsdom && pnpm test:ci:browser",
"test:ci": "pnpm lint && pnpm typecheck && pnpm test:ci:jsdom && pnpm test:ci:browser",
"test:ci:jsdom": "vitest run -c ci/vitest.config.js",
"test:ci:browser": "vitest run -c ci/vitest.config.browser.js",
"jsdoc": "jsdoc-cli-wrapper -c jsdoc.json .",
Expand Down

0 comments on commit fdf3d58

Please sign in to comment.