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