[RFC] Join all commands under an umbrella command `coverde`

mrverdant13 / coverde

A set of commands for coverage trace files manipulation.

MIT License

30 stars 11 forks source link

[RFC] Join all commands under an umbrella command `coverde` #16

Closed kranfix closed 3 years ago

kranfix commented 3 years ago

The current trend in Dart CLI is to have a a single ring to rule them all and the currently, this project has two commands: cov and x.

Why not to have a coverde command with the following API:

coverde filter  
coverde value  
coverde rm

If you agree, I would like to send the PR.

mrverdant13 commented 3 years ago

Hey, @kranfix !

That was exactly the initial idea but since the rm command is not strictly related to coverage functionality I decided to isolate it within a "generic actions" parent command (the x command).

However, rm is currently the only command with this cov-independent behavior. Thus, I guess it's just ok to use a single parent command for now as you suggest.

I'll keep an eye on new PRs in case you submit one. Otherwise, I'll be working on this change in the following days.

mrverdant13 commented 3 years ago

If new generic commands are included in future releases, it would be handy to nest them within a parent command. Let's cross that bridge when we come to it.