-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
Use a better arg for FINDSTR when using the m,l command in Windows. #887
Conversation
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.
I like the simplicity of this fix.
I noticed, however, that the output for directories is different after this commit. Any way to make sure we're listing the metadata about the directory itself?
I forgot to check directories. To be honest, what shows in |
Taking what we get from
what do you think of this transformation?
|
That looks fine to me. Can we be sure that the output of this command will remain stable? I'm not so familiar with Windows utilities. |
@lifecrisis, I decided to go with the built-in Vim functions, Unfortunately, there is no Vim function equivalent to the |
…erdtree into fix_local_issue_on_m-l_#883 Conflicts: nerdtree_plugin/fs_menu.vim
Fixes #883.
The only reason for the
FINDSTR
is to remove the extra seven lines of boilerplate that theDIR
command thinks you need to know. Instead of "grepping" for the date (in its many different formats), we can more easily look for the filename itself.