laforge has submitted this change. ( https://gerrit.osmocom.org/c/pysim/+/39748?usp=email )
Change subject: pylint: personalization.py: fix E1135: permitted_len unsupported-membership-test ...................................................................... pylint: personalization.py: fix E1135: permitted_len unsupported-membership-test pre-empt this from coming up in patch I60ea8fd11fb438ec90ddb08b17b658cbb789c051: E1135: Value 'self.permitted_len' doesn't support membership test (unsupported-membership-test) pickermitted_len Change-Id: I0343f8dbbffefb4237a1cb4dd40b576f16111073 --- M pySim/esim/saip/personalization.py 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: laforge: Looks good to me, approved fixeria: Looks good to me, but someone else must approve Jenkins Builder: Verified diff --git a/pySim/esim/saip/personalization.py b/pySim/esim/saip/personalization.py index 61973e7..cd23944 100644 --- a/pySim/esim/saip/personalization.py +++ b/pySim/esim/saip/personalization.py @@ -112,7 +112,7 @@ key_id = None kvn = None key_usage_qual = None - permitted_len = None + permitted_len = [] def validate(self): if not isinstance(self.input_value, (io.BytesIO, bytes, bytearray)): -- To view, visit https://gerrit.osmocom.org/c/pysim/+/39748?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: merged Gerrit-Project: pysim Gerrit-Branch: master Gerrit-Change-Id: I0343f8dbbffefb4237a1cb4dd40b576f16111073 Gerrit-Change-Number: 39748 Gerrit-PatchSet: 3 Gerrit-Owner: neels <nhofm...@sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria <vyanits...@sysmocom.de> Gerrit-Reviewer: laforge <lafo...@osmocom.org>