Remove short-flags in `cli.flags` if they're defined as `alias`

const cli = m({
	argv: ['-c', '-l'],
	flags: {
		coco: {
			alias: 'c'
		},
		loco: {
			alias: 'l'
		}
	}
});

console.log(cli.flags);
//=> { c: true, coco: true, l: true, loco: true }

Since c and l are defined as alias for two other flags, I think we should just exclude the short-flags from the cli.flags output. They just create noise there.

So it would be this instead:

console.log(cli.flags);
//=> { coco: true, loco: true }

Thoughts?

picchikevin earned $40.00 by resolving this issue!

• Checkout the Issuehunt explorer to discover more funded issues.
• Need some help from other developers? Add your repositories on Issuehunt to raise funds.

[revelt]

hi all, around 10 meow dependent CLI’s of mine were dependent on aliases. I’ve been using if (cli.flags.z) all over the code for convenience. What was done was done, v6 is out and I had to fix bunch of CLI’s but in the future please don’t remove issues if they already exist and don’t block anything.

[sindresorhus]

I’ve been using if (cli.flags.z) all over the code for convenience.

Convenience for you, but worse readability for anyone else reading your code. The change was done exactly to fix code like yours.

please don’t remove issues if they already exist and don’t block anything.

I cannot promise that. If something will help people improve their CLIs or code, I’ll do it.