[QGIS-Developer] spellchek on Travis

[Matthias Kuhn]

Hi Alessandro,

Spellcheck also guarantees consistent use of names in public API we 
expose (setColor vs. setColour etc).

Bests

Matthias

On 1/23/20 8:52 AM, Alessandro Pasotti wrote:
> Thanks a lot Denis!
>
> But what's the point in checking the spelling of C++ code? It won't 
> build if a method/class/variable is mispelled.
>
> Isn't that possible to limit the spellcheck to strings (and comments) ?
>
>
>
> On Wed, Jan 22, 2020 at 10:44 AM Denis Rouzaud 
> <denis.rouzaud at gmail.com <mailto:denis.rouzaud at gmail.com>> wrote:
>
>     see
>     https://github.com/qgis/QGIS/blob/master/scripts/spell_check/spelling.dat#L1
>
>     fixed in
>     https://github.com/qgis/QGIS/commit/13356efb7eda360d7e12a39761110df0dab8eb55
>
>     Le mer. 22 janv. 2020 à 10:37, Alessandro Pasotti
>     <apasotti at gmail.com <mailto:apasotti at gmail.com>> a écrit :
>
>         Hi,
>
>         I find it very useful but sometimes it return false positives:
>
>         https://travis-ci.org/qgis/QGIS/jobs/640060885#L897
>
>         is there anything we can do to fix it?
>
>         -- 
>         Alessandro Pasotti
>         w3: www.itopen.it <http://www.itopen.it>
>         _______________________________________________
>         QGIS-Developer mailing list
>         QGIS-Developer at lists.osgeo.org
>         <mailto:QGIS-Developer at lists.osgeo.org>
>         List info: https://lists.osgeo.org/mailman/listinfo/qgis-developer
>         Unsubscribe:
>         https://lists.osgeo.org/mailman/listinfo/qgis-developer
>
>
>
> -- 
> Alessandro Pasotti
> w3: www.itopen.it <http://www.itopen.it>
>
> _______________________________________________
> QGIS-Developer mailing list
> QGIS-Developer at lists.osgeo.org
> List info: https://lists.osgeo.org/mailman/listinfo/qgis-developer
> Unsubscribe: https://lists.osgeo.org/mailman/listinfo/qgis-developer
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osgeo.org/pipermail/qgis-developer/attachments/20200123/ea691eaf/attachment.html>