2015-06-25 13:53 GMT+01:00 Marcel de Rooy <m.de.r...@rijksmuseum.nl>:
>> Maybe the script should not process this check if codespell is not
>> installed. Patch welcomed ;-)
>
> Sorry, but I would rather prefer that you wrote that patch ;)

Done. Now the script warn instead of die and mark the test "spelling"
as skipped.

>> It's in test for the moment, we will see if the number of false positive
>> is high or not. Why do you worry about them?
>
> We should spend our QA time on the code not the spelling of comments.

That's why it's good to make a script do the job :)
_______________________________________________
Koha-devel mailing list
Koha-devel@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-devel
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

Reply via email to