add --names option

[Riyabelle25]

solves #50, by adding the --names option.

.option('names', {
			alias: 'n',
			default: false,
			describe: 'Shows the list of repo names with their owner',
			type: 'boolean',
		})

On setting true it outputs the list of repos-with-their owner instead of the repo-report table.
will remove the alias if it's not required here, or can come up with a more meaningful name for the option

cc: @ljharb

[ljharb]

I think the const metrics = code can be moved down, to right above where the table is generated.

i then also wonder if maybe the code that gets, filters, and sorts repositories could live in a separate function that detail calls? and then, we could have repo-report --names call that directly instead of adding this to detail?

[ljharb]

What do you think about #67 (review) ?

[Riyabelle25]

I think it's o-kay for now.
Moving those functionalities to a separate utils file would defy the purpose of having detail.js, as it's most of the code from there (L105 - L122). It looks like it might complicate things for the sake of better modularity

[ljharb]

That's true that it's the bulk of the detail command - but the "generateDetailTable" call is really the point of that, getting repo names is incidental.

I'm happy to land this as-is, but I'll probably refactor it this way anyways :-)

[Riyabelle25]

Aight. Perhaps I can open a separate issue addressing this once the current PR is merged?

[ljharb]

Turns out we already had repo-report ls, but it'd been broken at some point, so I moved this code into that separate (re-created) command :-)

[Riyabelle25]

Woah! Just saw your refactored code 🙌