This is an automated email from the ASF dual-hosted git repository. shazron pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/cordova-android.git
The following commit(s) were added to refs/heads/master by this push: new 02ee925 Remove obsolete jshint comments (#444) 02ee925 is described below commit 02ee92510376b23476039e29e1ec5ffaf2102b83 Author: Raphael von der GrĂ¼n <raphine...@gmail.com> AuthorDate: Mon May 21 12:02:14 2018 +0200 Remove obsolete jshint comments (#444) --- bin/templates/cordova/lib/AndroidManifest.js | 4 ---- bin/templates/cordova/lib/AndroidStudio.js | 2 -- bin/templates/cordova/lib/builders/GenericBuilder.js | 2 -- bin/templates/cordova/lib/builders/GradleBuilder.js | 2 -- bin/templates/cordova/lib/check_reqs.js | 2 -- bin/templates/cordova/lib/emulator.js | 2 -- bin/templates/cordova/lib/pluginHandlers.js | 2 -- bin/templates/cordova/lib/retry.js | 2 -- bin/templates/cordova/lib/run.js | 2 -- bin/templates/project/assets/www/cordova.js | 2 -- test/app/src/main/assets/www/cordova.js | 2 -- 11 files changed, 24 deletions(-) diff --git a/bin/templates/cordova/lib/AndroidManifest.js b/bin/templates/cordova/lib/AndroidManifest.js index 5b7077a..98f8202 100644 --- a/bin/templates/cordova/lib/AndroidManifest.js +++ b/bin/templates/cordova/lib/AndroidManifest.js @@ -51,15 +51,11 @@ AndroidManifest.prototype.setVersionCode = function (versionCode) { }; AndroidManifest.prototype.getPackageId = function () { - /* jshint -W069 */ return this.doc.getroot().attrib['package']; - /* jshint +W069 */ }; AndroidManifest.prototype.setPackageId = function (pkgId) { - /* jshint -W069 */ this.doc.getroot().attrib['package'] = pkgId; - /* jshint +W069 */ return this; }; diff --git a/bin/templates/cordova/lib/AndroidStudio.js b/bin/templates/cordova/lib/AndroidStudio.js index 68d4d8f..e04d8a8 100644 --- a/bin/templates/cordova/lib/AndroidStudio.js +++ b/bin/templates/cordova/lib/AndroidStudio.js @@ -4,8 +4,6 @@ * @param {String} root Root folder of the project */ -/* jshint esnext: false */ - module.exports.isAndroidStudioProject = function isAndroidStudioProject (root) { return true; }; diff --git a/bin/templates/cordova/lib/builders/GenericBuilder.js b/bin/templates/cordova/lib/builders/GenericBuilder.js index 892aa38..33c40ea 100644 --- a/bin/templates/cordova/lib/builders/GenericBuilder.js +++ b/bin/templates/cordova/lib/builders/GenericBuilder.js @@ -109,9 +109,7 @@ function findOutputApksHelper (dir, build_type, arch) { var archSpecific = !!/-x86|-arm/.exec(path.basename(ret[0])); // And show only arch-specific ones (or non-arch-specific) ret = ret.filter(function (p) { - /* jshint -W018 */ return !!/-x86|-arm/.exec(path.basename(p)) === archSpecific; - /* jshint +W018 */ }); if (archSpecific && ret.length > 1 && arch) { diff --git a/bin/templates/cordova/lib/builders/GradleBuilder.js b/bin/templates/cordova/lib/builders/GradleBuilder.js index 30d3c0f..669a35b 100644 --- a/bin/templates/cordova/lib/builders/GradleBuilder.js +++ b/bin/templates/cordova/lib/builders/GradleBuilder.js @@ -246,9 +246,7 @@ GradleBuilder.prototype.prepEnv = function (opts) { // If it's not set, do nothing, assuming that we're using a future version of gradle that we don't want to mess with. // For some reason, using ^ and $ don't work. This does the job, though. var distributionUrlRegex = /distributionUrl.*zip/; - /* jshint -W069 */ var distributionUrl = process.env['CORDOVA_ANDROID_GRADLE_DISTRIBUTION_URL'] || 'https\\://services.gradle.org/distributions/gradle-4.4-all.zip'; - /* jshint +W069 */ var gradleWrapperPropertiesPath = path.join(self.root, 'gradle', 'wrapper', 'gradle-wrapper.properties'); shell.chmod('u+w', gradleWrapperPropertiesPath); shell.sed('-i', distributionUrlRegex, 'distributionUrl=' + distributionUrl, gradleWrapperPropertiesPath); diff --git a/bin/templates/cordova/lib/check_reqs.js b/bin/templates/cordova/lib/check_reqs.js index 4a22360..2ef5aa8 100644 --- a/bin/templates/cordova/lib/check_reqs.js +++ b/bin/templates/cordova/lib/check_reqs.js @@ -19,8 +19,6 @@ under the License. */ -/* jshint sub:true */ - var shelljs = require('shelljs'); var child_process = require('child_process'); var Q = require('q'); diff --git a/bin/templates/cordova/lib/emulator.js b/bin/templates/cordova/lib/emulator.js index 305e2e3..82106bb 100644 --- a/bin/templates/cordova/lib/emulator.js +++ b/bin/templates/cordova/lib/emulator.js @@ -19,8 +19,6 @@ under the License. */ -/* jshint sub:true */ - var android_versions = require('android-versions'); var retry = require('./retry'); var build = require('./build'); diff --git a/bin/templates/cordova/lib/pluginHandlers.js b/bin/templates/cordova/lib/pluginHandlers.js index b5a46cf..f57c283 100644 --- a/bin/templates/cordova/lib/pluginHandlers.js +++ b/bin/templates/cordova/lib/pluginHandlers.js @@ -14,8 +14,6 @@ * */ -/* jshint unused: vars */ - var fs = require('fs'); var path = require('path'); var shell = require('shelljs'); diff --git a/bin/templates/cordova/lib/retry.js b/bin/templates/cordova/lib/retry.js index c464b9d..3ea8f14 100644 --- a/bin/templates/cordova/lib/retry.js +++ b/bin/templates/cordova/lib/retry.js @@ -19,8 +19,6 @@ under the License. */ -/* jshint node: true */ - 'use strict'; var events = require('cordova-common').events; diff --git a/bin/templates/cordova/lib/run.js b/bin/templates/cordova/lib/run.js index b97fce2..183fc92 100644 --- a/bin/templates/cordova/lib/run.js +++ b/bin/templates/cordova/lib/run.js @@ -19,8 +19,6 @@ under the License. */ -/* jshint loopfunc:true */ - var path = require('path'); var build = require('./build'); var emulator = require('./emulator'); diff --git a/bin/templates/project/assets/www/cordova.js b/bin/templates/project/assets/www/cordova.js index 24400bd..3ae354c 100644 --- a/bin/templates/project/assets/www/cordova.js +++ b/bin/templates/project/assets/www/cordova.js @@ -22,8 +22,6 @@ var PLATFORM_VERSION_BUILD_LABEL = '7.2.0-dev'; // file: src/scripts/require.js -/* jshint -W079 */ -/* jshint -W020 */ var require; var define; diff --git a/test/app/src/main/assets/www/cordova.js b/test/app/src/main/assets/www/cordova.js index 1814e02..11da9eb 100644 --- a/test/app/src/main/assets/www/cordova.js +++ b/test/app/src/main/assets/www/cordova.js @@ -22,8 +22,6 @@ var PLATFORM_VERSION_BUILD_LABEL = '6.2.0-dev'; // file: src/scripts/require.js -/*jshint -W079 */ -/*jshint -W020 */ var require, define; -- To stop receiving notification emails like this one, please contact shaz...@apache.org. --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@cordova.apache.org For additional commands, e-mail: commits-h...@cordova.apache.org