Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Fixed some lint warnings #400

Merged
merged 1 commit into from Mar 7, 2013
Jump to file or symbol
Failed to load files and symbols.
+5 −5
Split
View
@@ -40,7 +40,7 @@ package.get = function (dir, options, callback) {
package.read(dir, function (err, pkg) {
if (err) {
if (err.toString() === "Error: Invalid package.json file") {
- jitsu.log.error(err.toString())
+ jitsu.log.error(err.toString());
return callback(
'Please make sure ' + (path.join(dir, '/package.json')).grey + ' is valid JSON',
false,
@@ -469,7 +469,7 @@ package.updateTarball = function (version, pkg, existing, firstSnapshot, callbac
if (!firstSnapshot) {
jitsu.log.silly('Existing version: ' + existing.version.magenta);
jitsu.log.silly('Local version: ' + pkg.version.magenta);
- var old = semver.gte(existing.version, pkg.version);
+ old = semver.gte(existing.version, pkg.version);
if (old) {
//
@@ -506,7 +506,7 @@ package.updateTarball = function (version, pkg, existing, firstSnapshot, callbac
//
// Default to build if user inputs as -r arg something ridiculous
//
- if (pkg.version == null) {
+ if (pkg.version === null) {
pkg.version = semver.inc(existing.version, jitsu.config.get('release') || 'build');
}
}
@@ -706,7 +706,7 @@ function searchStartScript(dir) {
scripts.push(i + '.js', i + '.coffee');
});
- for (i in scripts) {
+ for (var i in scripts) {
if (existsSync(path.join(dir, scripts[i]))) {
return scripts[i];
}
@@ -786,7 +786,7 @@ function fillPackage (base, dir, callback) {
// TODO: Something here...
//
jitsu.log.error('Unable to add properties to package description.');
- jitsu.log.error(require('util').inspect(err));
+ jitsu.log.error(util.inspect(err));
return callback(err);
}