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

File mimetype filtering #10

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

giulio-Joshi
Copy link

Greetings,
I'd be my pleasure if you could evaluate this little modification to the script behavior:

  • File mimetype is evaluated with file command, leaving non-text file out of the selection
  • Added DEBUG env variable to check script internals&commands
  • Validated method with shellcheck utility, improving from for+find
  • added zipfile in /drill/ as example of file not to be touched

I arrived later after #9 (nice job!),
I'll be happy to handle conflicts on my side in case this modification is valuable to your project.

thanks you 🙇🏼

giulio-Joshi added 2 commits October 13, 2022 21:59
* Added DEBUG env variable to check script internals&commands

* Validated method with `shellcheck` utility, improving from `for`+`find`

* added zipfile in `/drill/` as example of file not to be touched
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.

1 participant