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 exclude file from scan feature #253
Conversation
* Abort exclusion instead of panicking when pattern isn't valid
will the exclude file follow gitignore styling or some kind of YAML configuration in some way? |
It is in the config file as an extra option |
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.
One suggested behaviour change and some minor cosmetic stuff.
* changed behavior of exclude function to continue and ignore invalide regex patterns * added some more tests (windows networks and continue after regex error)
One last QA check before we merge this. |
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.
More cosmetic fixes.
From testing, I think the exclusions should also apply when performing a clean. Currently, it does not apply these. |
it's supposed to be only for the Scan, i'll create a separate PR with an option for the Clean |
I think i changed everything needed , not 100% sure on the tsx formatting i did it manually. |
* Added exclude file from scan feature * Abort exclusion instead of panicking when pattern isn't valid * Added UI configuration for exclude patterns * * cosmetic fixes * changed behavior of exclude function to continue and ignore invalide regex patterns * added some more tests (windows networks and continue after regex error)
Added a basic regexp pattern matching file exclusion.
You can add the patterns in the config file or directly from the UI
For the config file
the first excludes all files ending in
sample.mp4
the second hidden directories
/.directory/
the third should work for windows based directories (the \ needs double escaping)
and so on
If added through the UI you don't need to escape the
\
character