krebs / krops

Mirror of https://cgit.krebsco.de/krops/about/ (PRs / issues accepted, as can be seen by not having them disabled)
Do What The F*ck You Want To Public License
135 stars 18 forks source link

support for include filters #15

Closed ghost closed 4 years ago

ghost commented 4 years ago

I am using this to solve the problem described in #14. I kept the old exclude option to prevent breaking existing setups. It was not possible to just add an include option, because the order of the parameters is important.

Mic92 commented 4 years ago

Maybe the old --exclude option could be deprecated using a warning message?

4z3 commented 4 years ago

Maybe the old --exclude option could be deprecated using a warning message?

Yes, we'll do that, but no hurry (from my side at least) as the old and new options don't really conflict each other.