Timo Rößner
Timo Rößner
I think this is a fantastic idea well worth pursuing.
Oh, good catch! Yes, that definitely looks like a bug on our side. Thank you for that detailed report, we will look into this (not sure when though).
Hmmm, to me it seems like this is one of the edge cases where we'll never have a solution that works for everybody. I agree with @mvz that the check...
I'm blocked the next weeks unfortunately, would anybody else like to tackle this?
The bad news: Unfortunately, this is not possible at the moment. Neither does Reek offer the option to use more than one configuration file nor does it allow you to...
Not entirely sure I follow, couldnt you just add those exceptions yourself using the `ignore_iterators` configuration described [here](https://github.com/troessner/reek/blob/master/docs/Nested-Iterators.md)?
Currently `ignore_iterators` works on methods alone. Whitelisting methods like `create` and so on would defeat the purpose of the whole smell detector since this means it would also not report...
Thanks for extracting that. I think we should really make an effort to have multiple examples for every detector even though fixing smells is highly dependant on your domain logic...
I also think that we expand the docs for [SimulatedPolymorphism](https://github.com/troessner/reek/blob/master/docs/Simulated-Polymorphism.md) significantly. My grandmother should be able to understand why this is bad and right now our explanation leaves a lot...
Fully agree, right now this set up is a bit confusing.