-
-
Notifications
You must be signed in to change notification settings - Fork 102
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
SearchMethod should be IEnumerable #25
Comments
Hi Alexandre, the simple answer is no. |
@chrissainty what's the impact since |
@AlanCS I never said there was a problem, it's time. I'm very busy and this is very small and low priority change, I'll make it when I can. But please feel free to make the change and create a PR if you would like to see it dealt with quicker. |
Is there any reason that SearchMethod needs to be List ? IEnumerable is the best choice to collection parameters.
The text was updated successfully, but these errors were encountered: