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

Exact path matching command line option #980

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

SeeStarz
Copy link

@SeeStarz SeeStarz commented Feb 8, 2025

Proof of concept feature improvement related to #260

This adds a new option to allow exact matching of sub path (the ones in between slashes). e.g. zoxide query rust matches rust but not rust-analyzer. However I haven't been able to understand how to integrate this option into CLI options, as such this pull request isn't complete yet.

I personally think just being able to exact match directories would help with QoL especially since the ones most vulnerable to regex matching is short names anyways.

Any feedback would be welcome!

@SeeStarz SeeStarz changed the title Stream filter logic Stream filter by exact sub path Feb 8, 2025
@SeeStarz
Copy link
Author

Now that it comes to shell integration I don't think I am qualified to progress any further, I do hope this will be somewhat useful if anyone has similar ideas.

The added feature is -e or --exact flag on zoxide query

@dev-ardi
Copy link

Your git history is whack since you merged instead of rebasing

@SeeStarz
Copy link
Author

I completely missed that, thanks!
I'm not sure where I f'd up but nothing a good old force push can't solve.
Note: If anyone has seen my PR before, both commits are merged into one forming a complete feature (albeit missing shell integration)

@SeeStarz SeeStarz marked this pull request as ready for review February 17, 2025 12:47
@SeeStarz SeeStarz changed the title Stream filter by exact sub path Exact path matching command line option Feb 17, 2025
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