donejs / deploy

Deployment utilities for DoneJS
MIT License
4 stars 1 forks source link

glob should not be required #8

Closed matthewp closed 9 years ago

matthewp commented 9 years ago

Currently it throws an exception if you leave out glob. The reason is _filePatterns will return [undefined] if you don't have a glob in your config. https://github.com/donejs/deploy/blob/master/lib/package.js#L9

mjstahl commented 9 years ago
exports._filePatterns = function(config) {
        if (config.blob) {
                if (_.isArray(config.glob)) {
                        return config.glob;
                }
                return [config.glob];
        }
    return [];
};
matthewp commented 9 years ago

looks right to me.