[Seaside] ignore Code Critics rules per class.

Lukas Renggli renggli at gmail.com
Fri Aug 7 14:08:21 UTC 2009


> Thanks, that's got it. So I put the pragma in any one method for a class
> rule, correct?

Yes, any method will do. Unfortunately Squeak/Pharo has no possibility
to annotate other entities than methods.

Lukas

> Is there a place to attach the pragma to the class itself?
>
> Thanks, ...Stan
>
>
>
> Lukas Renggli wrote:
>>
>>> Hi, When I subclass WASession, I get 'Class not referenced' from code
>>> critics; there's one instance of the problem for each method in the
>>> class.
>>> I've tried to set a pragma to ignore the rule as per:
>>> http://www.lukas-renggli.ch/blog/ignoring-lint-rules , but can't get it
>>> to
>>> ignore:
>>>
>>> currentNeatSheet
>>>
>>>        <ignoreLintRule: 'Class not referenced' rationale: 'session class
>>> configured by Seaside, accessed by self session' author: 'Stan Shepherd'>
>>>
>>>        ^ currentNeatSheet
>>>
>>> or
>>>
>>> currentNeatSheet
>>>
>>>        <ignoreLintRule: 'RBClassNotReferencedRule' rationale: 'session
>>> class
>>> configured by Seaside, accessed by self session' author: 'Stan Shepherd'>
>>>
>>>        ^ currentNeatSheet
>>>
>>> or
>>>
>>> currentNeatSheet
>>>
>>>        <lint: 'RBClassNotReferencedRule' rationale: 'session class
>>> configured by
>>> Seaside, accessed by self session' author: 'Stan Shepherd'>
>>>
>>>        ^ currentNeatSheet
>>
>> All your examples should actually work. The reasons they don't is
>> because the filters only work on SelectorEnviornments. The
>> SelectorEnvironment is the object that is used internally to collect
>> the matches if the result are individual methods. In the case of the
>> RBClassNotReferencedRule a class environment is used, as the result
>> are individual classes.
>>
>> I tried to fix the problem in Refactoring-Core-lr.53. It might not be
>> perfect yet, but it should solve your problem.
>>
>> Lukas
>>
>> --
>> Lukas Renggli
>> http://www.lukas-renggli.ch
>> _______________________________________________
>> seaside mailing list
>> seaside at lists.squeakfoundation.org
>> http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside
>>
>>
>
> --
> View this message in context: http://www.nabble.com/Slime-in-one-click-image-tp24736487p24864271.html
> Sent from the Squeak - Seaside mailing list archive at Nabble.com.
>
> _______________________________________________
> seaside mailing list
> seaside at lists.squeakfoundation.org
> http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside
>

-- 
Lukas Renggli
http://www.lukas-renggli.ch


More information about the seaside mailing list