> On 31 Jul 2015, at 16:00, Paul DeBruicker <pdebr...@gmail.com> wrote: > > It seems like its not testing for the right conditions any more and therefore > not finding any messages that are implemented but not sent. > > > I think this and the problem I was having the other day is from removing the > use of the RBSmallLintContext class from checking criitcs issues. Why did > you decide to do that?
This happened during case 15724[1]. Shouldn’t be hard to fix but requires tests. As for context removal: Because we have 100500 contexts in Pharo and everyone uses it’s own. Another thing is that “why should every rule care about the context?”. I wan to check a single class, and the rule needs some kind of context. I don’t have a context I have a class. Another issues that we still have is that you cannot see whether your method/class is violating a rule. You have to go to a result, which is a context, query something… [1]: https://pharo.fogbugz.com/f/cases/15724/