Do not add CNAME grunt warning for manual deployment type

This prevents `grunt.warning` for missing CNAME if `options.deployType` is set to `manual`. 

As `manual` is only uploading a new version to the application there is no need to provide a CNAME.
Good for me, manual was introduced later as a deploy type so this check was
supposed to happen when manual did not exist and is probably a leftover.

Simone


On Thu, Aug 7, 2014 at 9:25 PM, Eric Caron <notifications@github.com> wrote:

> In tasks/awsebtdeploy.js:
>
> > @@ -481,8 +481,12 @@ module.exports = function (grunt) {
> >              var env = findEnvironmentByCNAME(data, options.environmentCNAME);
> >
> >              if (!env) {
> > -              grunt.log.error();
> > -              grunt.warn('Environment with CNAME "' + options.environmentCNAME + '" does not exist');
> > +              if (options.deployType === 'manual') {
> > +                grunt.log.write('Environment with CNAME "' + options.environmentCNAME + '" does not exist but is not required for manual deployment');
>
> Would it make more sense to not even have this check happen & not have any
> output when we hit this & are deployType manual? It seems like there's no
> point in showing the message to the user at all.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/simoneb/grunt-awsebtdeploy/pull/8/files#r15957774>.
>