# HG changeset patch # User Andrew Azores # Date 1508361167 14400 # Node ID caaf7994d26b93f07604783a4f8fb64f3283e69c # Parent 9906199e8f0f5aa350f8870955fe7fc6b5fe4dbf Reorder build targets Previous build:skiptests patch caused "clean" script to be run after "lint" script, with the net end result being that checkstyle coverage would never be available. Reviewed-by: jkang Review-thread: http://icedtea.classpath.org/pipermail/thermostat/2017-October/025466.html diff -r 9906199e8f0f -r caaf7994d26b package.json --- a/package.json Wed Oct 18 14:19:15 2017 -0400 +++ b/package.json Wed Oct 18 17:12:47 2017 -0400 @@ -108,10 +108,11 @@ "lint-itests": "eslint -c .eslintrc.test.yaml integration-test --ext .spec.js || npm run lint-itest-checkstyle", "lint-itests-checkstyle": "eslint -c .eslintrc.test.yaml --ext .spec.js -f checkstyle -o checkstyle/checkstyle-itest.xml src", "lint": "npm run lint-ts && npm run lint-js && npm run lint-html", - "prebuild": "npm run license-check && npm run lint", - "build": "npm run build:skiptests", + "prebuild": "npm run clean && npm run license-check && npm run lint", + "build": "npm run build:compile", + "build:compile": "webpack --bail --progress --profile", "prebuild:skiptests": "npm run clean", - "build:skiptests": "webpack --bail --progress --profile", + "build:skiptests": "npm run build:compile", "postbuild": "npm test", "stop-mockapi": "forever stop --silent mock-api/mockapi.server.js || true", "prestart-mockapi": "npm run stop-mockapi",