build: Merge doc linting into 'npm test'
authorTimo Tijhof <krinklemail@gmail.com>
Wed, 18 Mar 2020 18:19:39 +0000 (18:19 +0000)
committerTimo Tijhof <krinklemail@gmail.com>
Wed, 18 Mar 2020 18:38:49 +0000 (18:38 +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 352dd13..d349f54 100644 (file)
@@ -1,7 +1,7 @@
 {
   "private": true,
   "scripts": {
-    "test": "grunt test",
+    "test": "grunt test && npm run doc",
     "qunit": "grunt qunit",
     "doc": "jsduck",
     "postdoc": "grunt copy:jsduck",