[MapProxy-dev] RFC: Introduce linting

[Johannes Weskamm]

+1


Greetings,

Johannes

Am 13.01.21 um 08:36 schrieb Julien R:
> Hello,
>
> I would like to propose adding linting to the mapproxy codebase and
> enforce it on the CI. The benefits of linting are mostly that they
> make reviews easier because reviewers don't have to point stylistic
> issues.
>
> Johannes Weskamm originally proposed to introduce linting in this
> github issue:
> https://github.com/mapproxy/mapproxy/issues/478
>
> I personally have experience with using flake8 professionally for
> multiple years and it works great. I started experimenting with
> setting up flake8 on the CI here:
> https://github.com/mapproxy/mapproxy/pull/486
>
> If this is accepted, the plan would be to:
> - Introduce flake8 on CI, but disable all errors at first
> - Enable errors one by one in separate commits
>
> This is to avoid submitting a massive PR touching everything. See the
> attached 'mapproxy_flake8_stats.txt' file for statistics about the
> "errors" found by flake8.
>
> Best regards,
> Julien
>
> --
> Julien Rebetez
> Lead Software & Machine Learning Engineer
> Picterra SA
> Rue de la Mouline 8, 1022 Chavannes, Switzerland
> www.picterra.ch
> <https://picterra.ch/?utm_source=signature&utm_medium=email&utm_campaign=43989-en-pt-email%20signature&utm_content=link>
>
> _______________________________________________
> MapProxy-dev mailing list
> MapProxy-dev at lists.osgeo.org
> https://lists.osgeo.org/mailman/listinfo/mapproxy-dev

-- 
  Dipl.-Geogr. Johannes Weskamm
  — Anwendungsentwickler —

  terrestris GmbH & Co. KG
  Kölnstraße 99
  53111 Bonn

  Tel: +49 (0)228 / 96 28 99 -555
  Fax: +49 (0)228 / 96 28 99 -57

  Email: weskamm at terrestris.de
  Web: https://www.terrestris.de

  Amtsgericht Bonn, HRA 6835
  Komplementärin: terrestris Verwaltungsgesellschaft mbH
  vertreten durch: Torsten Brassat, Marc Jansen
  
  Informationen über Ihre gespeicherten Daten finden Sie auf
  unserer Homepage unter folgendem Link: 
  https://www.terrestris.de/datenschutzerklaerung/ 

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osgeo.org/pipermail/mapproxy-dev/attachments/20210113/fba1f5c8/attachment.html>