-
Notifications
You must be signed in to change notification settings - Fork 44
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
Should package search be case sensitive? #327
Comments
I like this idea, do you want to look into it? |
Yes, I will look into it and submit a PR and we can evaluate from there! |
One thing to keep in mind: ideally we would push the searching down to dnf eventually, rather than do it ourselves. So let's check that dnf supports whatever features we add, so that such a refactor would be transparent to the user. |
I think performing case insensitive search may actually bring us closer to dnf's behavior: https://dnf.readthedocs.io/en/latest/command_ref.html#search-command
|
Perfect! Thanks for checking :) |
Most package names are all lower case, but some package names do have capital letters. Consider the package
NetworkManager
. At the moment making a request fornetworkmanager
will not yieldNetworkManager
in the response. I propose that the package search should be case insensitive.The text was updated successfully, but these errors were encountered: