build: Merge doc linting into 'npm test'
authorTimo Tijhof <krinklemail@gmail.com>
Wed, 18 Mar 2020 18:19:39 +0000 (18:19 +0000)
committerKrinkle <krinklemail@gmail.com>
Wed, 18 Mar 2020 18:36:44 +0000 (18:36 +0000)
Whether JSDuck or JSDoc3, it's good to verify that there are no
regressions in the doc syntax. This has been enforced by WMF CI
for many years with a dedicated Jenkins job.

However, both 'grunt lint' and 'npm run doc' take a relatively
small amount of time in CI:

* grunt lint: ~ 35s (not incl 'npm install')
* npm run doc: ~ 10s (not incl 'npm install')

Change-Id: If22b7bc64266e43088c7dec8138d81c938687fb9

package.json

index b2fc93b..46f6a58 100644 (file)
@@ -2,7 +2,7 @@
   "private": true,
   "scripts": {
     "build": "grunt minify",
-    "test": "grunt lint",
+    "test": "grunt lint && npm run doc",
     "qunit": "grunt qunit",
     "doc": "jsduck",
     "postdoc": "grunt copy:jsduck",