Maybe the script should not process this check if codespell is not installed. Patch welcomed ;-)
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? On 25 Jun 2015 09:06, "Marcel de Rooy" <m.de.r...@rijksmuseum.nl> wrote: > There is no package for codespell on Wheezy. > But I would like to keep using the qa script :) > > I am not sure if we should check spelling in QA and/or spend time on false > negatives too.. > > -----Oorspronkelijk bericht----- > Van: koha-devel-boun...@lists.koha-community.org [mailto: > koha-devel-boun...@lists.koha-community.org] Namens Jonathan Druart > Verzonden: donderdag 18 juni 2015 16:34 > Aan: koha-devel@lists.koha-community.org > Onderwerp: [Koha-devel] QA script - New check for spelling > > Hi devs, > > QAing bug 14383, I have just added a new check in the QA script to check > the spelling in the code comments. > Let me know if you find any issues. > You will have to install codespell (apt-get install codespell). > > Cheers, > Jonathan > _______________________________________________ > 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/ >
_______________________________________________ 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/
