Skip to content

Commit

Permalink
Remove usage of this._ - only using modules now
Browse files Browse the repository at this point in the history
  • Loading branch information
SBoudrias committed Jan 6, 2015
1 parent d4f5ebc commit 30ed576
Showing 1 changed file with 4 additions and 3 deletions.
7 changes: 4 additions & 3 deletions lib/base.js
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,7 @@ var path = require('path');
var events = require('events');
var assert = require('assert');
var _ = require('lodash');
var _s = require('underscore.string');
var async = require('async');
var findup = require('findup-sync');
var chalk = require('chalk');
Expand Down Expand Up @@ -336,10 +337,10 @@ Base.prototype.parseOptions = function () {
Base.prototype.checkRequiredArgs = function () {
// If the help option was provided, we don't want to check for required arguments,
// since we're only going to print the help message anyway.
if (this.options.help) { return; }
if (this.options.help) return;

// Bail early if it's not possible to have a missing required arg
if (this.args.length > this._arguments.length) { return;}
if (this.args.length > this._arguments.length) return;

this._arguments.forEach(function (arg, position) {
// If the help option was not provided, check whether the argument was required,
Expand Down Expand Up @@ -524,7 +525,7 @@ Base.prototype.hookFor = function hookFor(name, config) {
// add the corresponding option to this class, so that we output these hooks
// in help
this.option(name, {
desc: this._.humanize(name) + ' to be invoked',
desc: _s.humanize(name) + ' to be invoked',
defaults: this.options[name] || ''
});

Expand Down

0 comments on commit 30ed576

Please sign in to comment.