We had another idea yesterday with Julien. Add a rule to check that in a test `self skip` is only present in the first statement of a test case. On Fri, Aug 17, 2018 at 9:38 AM Marcus Denker <marcus.denker@inria.fr> wrote:

...

On 13 Aug 2018, at 14:59, Lionel Akue <lionelakue@gmail.com> wrote:

Hello,

Do you have a rule you would like to get implement in Pharo?

I am reading about Renraku and I would like to exercise.

Thanks,

Lionel

Some ideas:

- Test Class not in Package with name -Tests - add rule: non-exisitng selector send in a method that has itself no senders means that the whole method most likely is dead code

Improvements:

- SharedPool subclasses should not trigger “excessive number of vars” QA rule (as their duty is to define variables)

Marcus

-- Guille Polito Research Engineer Centre de Recherche en Informatique, Signal et Automatique de Lille CRIStAL - UMR 9189 French National Center for Scientific Research - *http://www.cnrs.fr <http://www.cnrs.fr>* *Web:* *http://guillep.github.io* <http://guillep.github.io> *Phone: *+33 06 52 70 66 13