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

Change subject: utils: Add function to verify Luhn check digits and to sanitize 
ICCIDs
......................................................................


Patch Set 1:

(1 comment)

File tests/test_utils.py:

Robot Comment from checkpatch (run ID jenkins-gerrit-lint-14478):
https://gerrit.osmocom.org/c/pysim/+/36009/comment/930018f8_33eb9a92
PS1, Line 251:         # 19 digits without correct luhn; we expect chck digit 
to be added
'chck' may be misspelled - perhaps 'check'?



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

Gerrit-Project: pysim
Gerrit-Branch: master
Gerrit-Change-Id: I7812420cf97984dd834fca6a38c5e5ae113243cb
Gerrit-Change-Number: 36009
Gerrit-PatchSet: 1
Gerrit-Owner: laforge <[email protected]>
Gerrit-CC: Jenkins Builder
Gerrit-Comment-Date: Sun, 18 Feb 2024 18:49:38 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment

Reply via email to