Attention is currently required from: dexter.

osmith has posted comments on this change by dexter. ( 
https://gerrit.osmocom.org/c/pysim/+/42733?usp=email )

Change subject: cosmetic: remove excess linebreaks
......................................................................


Patch Set 1:

(1 comment)

Commit Message:

https://gerrit.osmocom.org/c/pysim/+/42733/comment/ebcc38d7_40bcf949?usp=email :
PS1, Line 10: difficult to maintain consistence with that scheme.
Note that PEP 8 says:

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

https://peps.python.org/pep-0008/#blank-lines

We use `ruff format` in some projects to have that consistently, e.g. testenv 
in osmo-ttcn3-hacks.



--
To view, visit https://gerrit.osmocom.org/c/pysim/+/42733?usp=email
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings?usp=email

Gerrit-MessageType: comment
Gerrit-Project: pysim
Gerrit-Branch: master
Gerrit-Change-Id: I2272abcdbba0e57c5118b69bc9398cd909a469b5
Gerrit-Change-Number: 42733
Gerrit-PatchSet: 1
Gerrit-Owner: dexter <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <[email protected]>
Gerrit-CC: osmith <[email protected]>
Gerrit-Attention: dexter <[email protected]>
Gerrit-Comment-Date: Fri, 08 May 2026 08:15:01 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No

Reply via email to