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

[sindresorhus]

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.

[IssueHuntBot]

@issuehuntfest has funded $40.00 to this issue. See it on IssueHunt

[IssueHuntBot]

@PicchiKevin has submitted a pull request. See it on IssueHunt

[IssueHuntBot]

@sindresorhus has rewarded $36.00 to @PicchiKevin. See it on IssueHunt

• 💰 Total deposit: $40.00
• 🎉 Repository reward(0%): $0.00
• 🔧 Service fee(10%): $4.00

[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.