> 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/

Reply via email to