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

Ignore options feature #83

Merged
merged 3 commits into from Apr 3, 2015
Merged

Ignore options feature #83

merged 3 commits into from Apr 3, 2015

Conversation

ajdaniel
Copy link
Contributor

A new feature to allow the user to supply terms to TVRenamer, which are compared against filenames when added to the list, and should any match, the file will be unchecked automatically. Useful to automatically ignore certain file types, such as .nfo, or additional files usually found accompanying TV shows, like samples.

Allows the user to choose keywords (default: ‘sample’) to dictate
ignored paths and filenames. Useful to ignore files like .nfo or sample
files.
@ajdaniel
Copy link
Contributor Author

@vipuldelwadia or @daveharris I'd rather one of you guys accept this merge please. I don't want to do it and screw up 😄

@ajdaniel
Copy link
Contributor Author

ajdaniel commented Apr 2, 2015

Oh cool Github put the new commit into my merge request. @davekeane if you are happy with that could you accept the merge request please?

@davekeane
Copy link
Member

Everything I've seen is good, but unfortunately I don't have a working dev environment to pull/test/review in. So I'm not comfortable clicking the merge button....

@vipuldelwadia vipuldelwadia self-assigned this Apr 3, 2015
vipuldelwadia added a commit that referenced this pull request Apr 3, 2015
@vipuldelwadia vipuldelwadia merged commit 154345b into master Apr 3, 2015
@vipuldelwadia
Copy link
Member

Tested locally, looks good, nice work!

@vipuldelwadia vipuldelwadia deleted the ignore_options_feature branch April 3, 2015 08:09
johnv02139 pushed a commit to johnv02139/tvrenamer that referenced this pull request Jun 18, 2018
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.

None yet

3 participants