Ah, missed this in the refactor to fix linter errors. Used to bind grunt as
this instead of using it as an arg. Nice catch.

On Feb 14, 2013, at 3:28 AM, hitsthings <notifications@github.com> wrote:

In tasks/lib/heroku-deploy.js:

> +
> +    getCurrentBranch(function(err, branch) {
> +      if (err) {
> +        return next(err);
> +      }
> +
> +      if (!branch) {
> +        console.log('No current branch');
> +
> +        getCurrentCommitHash(function(err, csid) {
> +          if (err) {
> +            return next(err);
> +          }
> +
> +          console.log('Using ' + csid + ' as ref to merge.');
> +          deployArgs[0].originRef = csid;

Haven't run it yet, but I would expect this and the occurrence below should
be deployArgs[1]. Looks like you're currently changing the grunt object.

—
Reply to this email directly or view it on
GitHub<https://github.com/hitsthings/grunt-heroku-deploy/pull/4/files#r3011532>.