Don't confuse jshint by talking about jshint

medic · Jun 28, 2018 · 2a43669 · 2a43669
1 parent 5ca201c
commit 2a43669
Show file tree

Hide file tree

Showing 4 changed files with 8 additions and 4 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,5 +1,9 @@
 # Changelog
 
+## v1.17.4
+* Don't confuse jshint by talking about jshint
+* Fix: first scheduled_task date is incorrect
+
 ## v1.17.3
 * [compile-app-settings] allow console.log() statements
 

diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "medic-conf",
-  "version": "1.17.3",
+  "version": "1.17.4",
   "description": "Configure Medic Mobile deployments",
   "main": "index.js",
   "scripts": {

diff --git a/src/lib/jshint-with-report.js b/src/lib/jshint-with-report.js
@@ -23,8 +23,8 @@ const DEFAULT_OPTS = {
 module.exports = (description, code, options) => {
   options = _.extend(options, DEFAULT_OPTS);
 
-  // jshint can't detect repeated declarations in the global scope, so we wrap
-  // our code in a function.  Ref: https://github.com/jshint/jshint/issues/3288
+  // wrap our code in a function because otherwise jshint can't detect repeated
+  // declarations.  Ref: https://github.com/jshint/jshint/issues/3288
   jshint(`(function() {
     ${code}
   }());`, options);