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 <laforge(a)osmocom.org>
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