-
Notifications
You must be signed in to change notification settings - Fork 12
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
Remove --full flag #13
Comments
@orta We thought about removing --full flag, but I have a small concern. non-git repositories still won't utilize improvements presented. Their search will still be slow. However, I do think we can still move --full as default option because I assume users using non-git repos will be much minority. What about adding a new flag like cc: @alloy @floere @segiddins |
Sounds good to me! |
Yeah, that's cool for me too |
Good then. |
After the improvements in #8 merged completely, I think we can get rid of
--full
flag because it would give answer in less than a second in most cases.The text was updated successfully, but these errors were encountered: