Patch Set 3: Code-Review+1

(1 comment)

if you must, but...
File src/osmo_gsm_tester/

Line 80:     def ready_for_pcu(self):
from a python doc point of view, nothing has changed at all. If a function is 
considered private, it should have an underscore. Otherwise whether test 
scripts call a function or not doesn't really change semantics of the class 
definition. I think there's no point to this :)

To view, visit
To unsubscribe, visit

Gerrit-MessageType: comment
Gerrit-Change-Id: Ie8978854404897ae5de7e3100d55d86c7a5c1df1
Gerrit-PatchSet: 3
Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Owner: Pau Espin Pedrol <>
Gerrit-Reviewer: Harald Welte <>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: Neels Hofmeyr <>
Gerrit-HasComments: Yes

Reply via email to