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

Add globbing pattern matching for search_path #179

Closed
wants to merge 0 commits into from

Conversation

anya004
Copy link

@anya004 anya004 commented Oct 11, 2024

Fixes #165

This PR adds support for glob patterns, controlled by a new globbing flag. When enabled (-globbing), the tool will use glob pattern matching to locate files and directories.

I decided to implement a flag so that the user is aware of the behaviour in case they have some unconventional filenames.

When globbing is enabled, matched paths are dynamically added to the searchPath array for further processing.

@kehoecj kehoecj added hacktoberfest-accepted Valid PR Hacktoberfest PR hacktoberfest 🎃 Hacktoberfest 2024 waiting-on-maintainer-review PR is waiting to be reviewed and functionally tested by the maintainers labels Oct 11, 2024
@kehoecj
Copy link
Collaborator

kehoecj commented Oct 11, 2024

@anya004 Thank you for the PR! Implementation looks good. Can you please take a loot at the golangci-lint findings?

cmd/validator/validator.go Outdated Show resolved Hide resolved
cmd/validator/validator.go Outdated Show resolved Hide resolved
@kehoecj kehoecj added pr-action-requested PR is awaiting feedback from the submitting developer and removed waiting-on-maintainer-review PR is waiting to be reviewed and functionally tested by the maintainers labels Oct 21, 2024
@anya004 anya004 force-pushed the add-globbing branch 2 times, most recently from e7114ee to b18647a Compare October 27, 2024 06:53
@kehoecj
Copy link
Collaborator

kehoecj commented Oct 28, 2024

@anya004 Please resolve conflicts

@kehoecj
Copy link
Collaborator

kehoecj commented Oct 30, 2024

@anya004 I'd like to get you merged in before Hacktoberfest is over. Please take a look at the the merge conflicts

@anya004
Copy link
Author

anya004 commented Oct 30, 2024

@kehoecj oops, I pushed an empty set of changes while trying to fix the conflicts. Here is a new PR: #215

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hacktoberfest 🎃 Hacktoberfest 2024 hacktoberfest-accepted Valid PR Hacktoberfest PR pr-action-requested PR is awaiting feedback from the submitting developer
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FEATURE REQUEST] Add support for glob patterns
2 participants