github/meteor
1
1
Fork 0
mirror of https://github.com/meteor/meteor.git synced 2026-05-02 03:01:46 -04:00
Code Issues Packages Projects Releases Wiki Activity

some error-checking for option definitions

Browse Source 
short options must be short, and you can't use names that mean something
already
This commit is contained in:
David Glasser
2014-02-04 18:06:22 -08:00
parent 7bf9d0a0d3
commit 42b5b76000
1 changed files with 4 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
tools/main.js
View File

@@ -47,11 +47,15 @@ var Command = function (options) {
_.extend(this, options);
_.each(this.options, function (value, key) {
if (key === "args" || key === "appDir")
throw new Error(options.name + ": bad option name " + key);
if (! _.has(value, 'type'))
value.type = String;
if (_.has(value, 'default') && _.has(value, 'required'))
throw new Error(options.name + ": " + key + " can't be both optional " +
"and required");
if (_.has(value, 'short') && value.short.length !== 1)
throw new Error(options.name + ": " + key + " has a bad short option");
});
};