-
Notifications
You must be signed in to change notification settings - Fork 108
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
Allow user to add to ignore_files list #71
Comments
Example folders/files to allow user to exclude:
|
This is a great idea. If you can come up with a PR it will be awesome. |
What would make this really powerful is to invert this enhancement to indicate what files you want to INCLUDE. The reason is there are directories that contain SOME images / movies you want, but you don't want to disturb the rest of the content. Listing what you want excluded will mean iterating on file types you don't want, running the script, seeing it took file types you didn't know were there, repeat. I propose including extensions that would be processed, and perhaps an option that says it will look for exif data in EVERY file, and if it finds it (regardless of extension), it will process it. Or "include all photo and video media" and your app maintains a config of extensions that encompasses. Don't know Python else I'd help. Love the work -- thank you so much. |
I'm working on this and want to make sure no one has any opposition to leveraging YAML for the configuration file. My intention is to introduce I also intend on allowing an alternate configuration file to be specified so we can go with Input/opinions welcome before I get too far down this path. Or thumbs up emojis is fine too. |
I don't mind having yaml config file. My only suggestion is to use |
Allow the user to append filenames to the
ignore_files
list.phockup/src/phockup.py
Line 13 in 4bac6db
Should allow the use of a flag, such as
-i
, to accept both:-i filename1.txt file2.docx
or-i filename1.txt -i file2.docx
)-i /path/to/phockupIgnoreList.txt
with contents ofphockupIgnoreList.txt
being:(The flag may need to differ for these two, such as using
-i
for in-line filename exclusions and-n
for passing an ignore file).This ignore feature should allow extension-level exclusions, such as
*.txt
as well as folder level exclusions (particularly for hidden folders), such as./.hidden/
The text was updated successfully, but these errors were encountered: