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.

Sign up for GitHub

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

Jump to bottom

--mtime-from-date breaks exec postprocessor (and probably others) #994

Closed
ImportTaste opened this issue Sep 14, 2020 · 0 comments
Closed

--mtime-from-date breaks exec postprocessor (and probably others) #994

ImportTaste opened this issue Sep 14, 2020 · 0 comments
Labels
enhancement

Comments

@ImportTaste
Copy link
Contributor

I noticed that when I was specifying --mtime-from-date in the command line, my postprocessor for exec in the config file wouldn't fire. Not sure why this is.

Specifying --exec in the command line works, but it's obviously more limited.
mikf added a commit that referenced this issue Sep 15, 2020
@mikf
implement 'config.accumulate()' (#994)
392d022
mikf added a commit that referenced this issue Sep 15, 2020
@mikf
accumulate postprocessor objects (#994)
231dd4c 
Instead of one 'postprocessors' setting overwriting all others lower
in the hierarchy, all postprocessors along the config path will now
get collected into one big list.

For example '--mtime-from-date' will therefore no longer cause
other postprocessor settings in a config file to get ignored.
@mikf mikf closed this as completed Sep 15, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@mikf @ImportTaste