Change in osmo-sim-auth[master]: Use 2 blank lines when needed

This is merely a historical archive of years 2008-2021, before the migration to mailman3.

A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/gerrit-log@lists.osmocom.org/.

rousseau gerrit-no-reply at lists.osmocom.org
Mon Oct 26 09:09:40 UTC 2020


rousseau has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-sim-auth/+/20897 )

Change subject: Use 2 blank lines when needed
......................................................................


Patch Set 1:

> it's news to me that there is a requirement for multiple new lines between functions.  But well, I always found python syntax rather odd...

It is not Python *requirement* but it is in PEP8 "the Style Guide for Python Code"
https://pep8.org/#blank-lines

> Surround top-level function and class definitions with two blank lines.

I used the tool pycodestyle to check the coding style and make it homogenous.
https://pypi.org/project/pycodestyle/


-- 
To view, visit https://gerrit.osmocom.org/c/osmo-sim-auth/+/20897
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-sim-auth
Gerrit-Branch: master
Gerrit-Change-Id: Ic1e902b2c112b9b485a090a591db7cdbabea77f4
Gerrit-Change-Number: 20897
Gerrit-PatchSet: 1
Gerrit-Owner: rousseau <ludovic.rousseau+osmocom at free.fr>
Gerrit-Reviewer: fixeria <vyanitskiy at sysmocom.de>
Gerrit-Reviewer: laforge <laforge at osmocom.org>
Gerrit-Comment-Date: Mon, 26 Oct 2020 09:09:40 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20201026/bf64c22d/attachment.htm>


More information about the gerrit-log mailing list