search

qooxdoo-archive / qooxdoo-cli

(deprecated, moved into qooxdoo-compiler) qx commandline
MIT License
4 stars 5 forks source link

Streamline options #35

Closed cboulanger closed 6 years ago

cboulanger commented 6 years ago

Options should only be displayed if they are really available in a command. I would therefore argue for not including any default options (not even --verbose). Instead, they should be listed, when implemented, in each command class, even if this leads to duplicate code. For example, they should be removed completely here: https://github.com/qooxdoo/qooxdoo-cli/blob/master/commands/contrib.js

hkollmann commented 6 years ago

Is it possible to have some sort of inheritance here?

cboulanger commented 6 years ago

@hkollmann There already is inheritance, but I'd like to avoid it since it could lead to wrong documentation if an option is inherited but not implemented. Plus I think often there are often different aspects to the same option name in different commands. So I'd rather duplicate option descriptions.

hkollmann commented 6 years ago

It's done with pr #61