Jenkins Builder has posted comments on this change by laforge. ( 
https://gerrit.osmocom.org/c/pysim/+/37848?usp=email )

Change subject: pySim.esim.saip: maintain a parsed fileystem hierarchy
......................................................................


Patch Set 1:

(2 comments)

File pySim/esim/saip/__init__.py:

Robot Comment from checkpatch (run ID jenkins-gerrit-lint-17790):
https://gerrit.osmocom.org/c/pysim/+/37848/comment/ac2a76c0_52463423?usp=email :
PS1, Line 1708:         """Create and link an apropriate FsNode for the given 
'file' and insert it.
'apropriate' may be misspelled - perhaps 'appropriate'?


Robot Comment from checkpatch (run ID jenkins-gerrit-lint-17790):
https://gerrit.osmocom.org/c/pysim/+/37848/comment/0e79dc69_1b55d655?usp=email :
PS1, Line 1724:         logger.debug("%s.add_file(%s) -> return cur_df=%s" % 
(self, file, cur_df))
trailing whitespace



--
To view, visit https://gerrit.osmocom.org/c/pysim/+/37848?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: I2caadc24b1087855f23f3c57cdf8dabbf81757c0
Gerrit-Change-Number: 37848
Gerrit-PatchSet: 1
Gerrit-Owner: laforge <[email protected]>
Gerrit-CC: Jenkins Builder
Gerrit-Comment-Date: Sat, 17 Aug 2024 13:41:52 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No

Reply via email to