New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
add common prefixes and subcommands to stats #773
add common prefixes and subcommands to stats #773
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Current dependencies on/for this PR: This comment was auto-generated by Graphite. |
1 similar comment
Current dependencies on/for this PR: This comment was auto-generated by Graphite. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Graphite is interesting 🤔
3af4cc8
to
2580338
Compare
Aww squash commits make it a lil sad :( |
@@ -93,3 +91,75 @@ impl Cmd { | |||
Ok(()) | |||
} | |||
} | |||
|
|||
// TODO: make this configurable? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
100%, though not blocking to merge this imo
Some people wanted
sudo
to be ignored, and then some commands likecargo
andgit
are so prevelant that the stat isn't interesting. This addresses both of those, although it's a little opinionated for now