move output information to printer

[cravler]

const { Command, Printer } = require('commander');

class CustomPrinter extends Printer {
 
    error(msg, ...args) {
        super.error(this.format(msg), ...args);
    }
 
    outputHelp(program) {
        return this.format(super.outputHelp(program));
    }
 
    format(str) {
        return [
            '\n', str.replace(/^/gm, '  '), '\n'
        ].join('');
    }
 
}

const program = new Command('example', new CustomPrinter());

program.version('0.0.1')
    .command('setup [env]')
    .description('application simple description', {
        env: 'env value'
    })
    .option('-f, --foo', 'enable some foo')
    .action(function(env, options) {});

program.parse(process.argv);
if (!program.args.length) program.help();

[cravler]

any comments?

[cravler]

Is this repository supported?

[abetomo]

The test has failed.
Could you check it?

[cravler]

any news?

[shadowspawn]

@cravler
What problem are you solving with this Pull Request?
Are there open Issues that this will resolve?

Or is this primarily an object separation suggestion, rather than a functional suggestion?

Edit: that probably sounded a bit blunt, sorry! I am working through backlog of Pull Requests making sure I understand them, so asking lots of questions.

[cravler]

This separation adds ability to write custom printers without touching the default. In my case I have Colorized Printer

[shadowspawn]

Thanks @cravler. This is quite interesting, but changes a lot of code. Offering customisation by inheritance is also more challenging for future maintenance than a callback.

I don't wish to add this at the moment. However we have a variety of requests around help and output. This may get revisited in the future if it can be used to resolve more issues.

Thank you for your contributions.