make deployment callable from other scripts with callback

[fivethreeo]

Example:

gulp.task('deploy_resource', function () {

    var format = require('string-format');
    var deploy = require(path.join(
      __dirname,
      'node_modules',
      'cfn-elasticsearch-domain',
      'node_modules',
      'cfn-lambda',
      'deploy'));

    return deploy(
      'eu-west-1', ['eu-west-1'], function (res) {
          console.log(format(res, 'eu-west-1'));
      }
    );
});

[bendavis78]

+1 - Better yet, why not export the deploy function in the top-level module?

gulp.task('deploy_resource', function() {
    var deploy = require('cfn-lambda').deploy;
    return deploy(...);
});

[fivethreeo]

Top level would make it impossible to use more than one cfn. If so it would need a module arg.

[fivethreeo]

Both the

"cfn-lambda-deploy": "node ./node_modules/cfn-lambda/deploy.js"

and

    var cfnlambda = require('cfn-lambda');
    var config = require('./config.json');
    var format = require('string-format');

    return cfnlambda.deploy(
      'cfn-elasticsearch-domain',
      'eu-west-1',
      ['eu-west-1'],
      function (res) {
          console.log(format(res, 'eu-west-1'));
      }
    );

Methods work.

[andrew-templeton]

If I drop a PR, comment again like this - I have over 25 public projects I'm sole contrib on and ~100 others I get notifs for. Chances are I liked your PR and just forgot about it :)

Thank you much for the commits! Really good stuff / +1

[fivethreeo]

Thank you :) This module really makes cfn lamdas easy. But could you do a new release in the npm registry? npm just wants to pull the one on there.

[andrew-templeton]

Going to release tonight