Skip to content
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

[Search directory] fix specifying base directories using quotes #163

Merged
merged 1 commit into from
May 28, 2021

Conversation

kidonng
Copy link
Contributor

@kidonng kidonng commented May 20, 2021

This was presented in the initial PR when the feature is added, but was (possibly mistakenly) broken by #137.

@PatrickF1
Copy link
Owner

How can I repro the bug?

@kidonng
Copy link
Contributor Author

kidonng commented May 21, 2021

I don't consider it a bug, though technically it did stop working.

To reproduce, just input 'with space/' and trigger file search. It won't scope the search to the folder.

@PatrickF1
Copy link
Owner

Hmm I was able to repro. But you know what? I vaguely remember changing unescaped_exp_token to just token b/c it broke something else. Now trying to remember what that was...

@kidonng
Copy link
Contributor Author

kidonng commented May 21, 2021

I vaguely remember changing unescaped_exp_token to just token b/c it broke something else.

If it breaks anything we should fix it instead. However this only affects the string match and I don't see any other issue.

@PatrickF1
Copy link
Owner

Did some more testing and it seems to not break anything. If it does introduce a new bug, I trust you to find it 😉

@PatrickF1 PatrickF1 changed the title [Search directory] match against unescaped token [Search directory] fix specifying base directories using quotes May 28, 2021
@PatrickF1 PatrickF1 merged commit 756513f into PatrickF1:main May 28, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants