Skip to content

Conversation

@ahundt
Copy link

@ahundt ahundt commented Aug 13, 2017

(#532)

@ahundt
Copy link
Author

ahundt commented Aug 13, 2017

Those filter classes like SimpleFilter seems to only support true and false options or I'd have used it. Hopefully my approach is reasonable but I'm open to make changes.

@jonathanslenders
Copy link
Member

Hi @ahundt,

Thanks a lot for this pull request.
The 1.0 branch is actually bugfix only, but I'd like to merge this in the 2.0 branch once I've had the time to review it.

It's very useful.

@ahundt
Copy link
Author

ahundt commented Aug 17, 2017

cool, thanks!

@ahundt
Copy link
Author

ahundt commented Aug 21, 2017

@jonathanslenders I've been using the existing functionality a bit more and I realized along with this change I'd also love for the selection of completion to be case-insensitive. In other words if there is a LICENSE.txt in a folder and you type lic<tab>, LICENSE.txt would be an option. I believe that would also be a very small change much like this one.

@anki-code
Copy link

@jonathanslenders this looks like merged and should be closed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants